372228e3b2f3e9968054bc00f367671282c4c58f: Bug 1416106 - Part 5: Implement base of keyframes graph. r?gl draft
Daisuke Akatsuka <dakatsuka@mozilla.com> - Fri, 19 Jan 2018 16:59:06 +0900 - rev 722579
Push 96176 by bmo:dakatsuka@mozilla.com at Fri, 19 Jan 2018 08:09:48 +0000
Bug 1416106 - Part 5: Implement base of keyframes graph. r?gl MozReview-Commit-ID: FpTZvREM35Y
a7dbd9ef2517e2019238f7943b8c6d196311dda7: Bug 1416106 - Part 4: Add test for property name component. r?gl draft
Daisuke Akatsuka <dakatsuka@mozilla.com> - Fri, 19 Jan 2018 16:57:38 +0900 - rev 722578
Push 96176 by bmo:dakatsuka@mozilla.com at Fri, 19 Jan 2018 08:09:48 +0000
Bug 1416106 - Part 4: Add test for property name component. r?gl MozReview-Commit-ID: En70sbOhcsr
7c7f96c44e71e111a1b15260b8f09785914a8143: Bug 1416106 - Part 3: Implement compositor warning. r?gl draft
Daisuke Akatsuka <dakatsuka@mozilla.com> - Fri, 19 Jan 2018 16:57:12 +0900 - rev 722577
Push 96176 by bmo:dakatsuka@mozilla.com at Fri, 19 Jan 2018 08:09:48 +0000
Bug 1416106 - Part 3: Implement compositor warning. r?gl MozReview-Commit-ID: 18qQ1FydML4
fd23e1b7ab30530c3b09e9c783c5da80cee36648: Bug 1416106 - Part 2: Implement compositor sign to property name. r?gl draft
Daisuke Akatsuka <dakatsuka@mozilla.com> - Fri, 19 Jan 2018 16:55:18 +0900 - rev 722576
Push 96176 by bmo:dakatsuka@mozilla.com at Fri, 19 Jan 2018 08:09:48 +0000
Bug 1416106 - Part 2: Implement compositor sign to property name. r?gl MozReview-Commit-ID: v7c6rt6vpl
4600c58b388c9521aa24c45e6908df6d6ce5b124: Bug 1416106 - Part 1: Implement base of property name. r?gl draft
Daisuke Akatsuka <dakatsuka@mozilla.com> - Fri, 19 Jan 2018 16:52:20 +0900 - rev 722575
Push 96176 by bmo:dakatsuka@mozilla.com at Fri, 19 Jan 2018 08:09:48 +0000
Bug 1416106 - Part 1: Implement base of property name. r?gl MozReview-Commit-ID: GSPIZG31RhB
d86b61059100fb5e9dc09520f31b982a1502d4b8: Bug 1431327 - Refactor setupStore helper function; r=bgrins. draft
Nicolas Chevobbe <nchevobbe@mozilla.com> - Wed, 17 Jan 2018 09:36:19 +0100 - rev 722574
Push 96175 by bmo:nchevobbe@mozilla.com at Fri, 19 Jan 2018 08:06:57 +0000
Bug 1431327 - Refactor setupStore helper function; r=bgrins. Change the signature of setupStore to accept an option object as its second parameter so it's more easy to remember what it does when reading the consumer code. Also, Pass the wrapped actions object to setupStore so we do use the same generator when adding messages later. Add more group messages in the group test to make sure group are being closed when a groupEnd message is passed in a messagesAdd batch. MozReview-Commit-ID: CBN0r8nBaAr
999bed89ad4cf729d03bb8f5b34786954e836312: Bug 1419226 - Part 5. Skip test_fullscreen-api.html and test_transitions_replacement_on_busy_frame.html. r?kat draft
Mantaroh Yoshinaga <mantaroh@gmail.com> - Fri, 19 Jan 2018 16:55:04 +0900 - rev 722573
Push 96174 by bmo:mantaroh@gmail.com at Fri, 19 Jan 2018 07:56:06 +0000
Bug 1419226 - Part 5. Skip test_fullscreen-api.html and test_transitions_replacement_on_busy_frame.html. r?kat If previous patch remove the timer which is firing a MozAfterPaint after 100ms, seteral webrender mohcitest are failed due to MozAfterPaint. This patch will skip these tests. MozReview-Commit-ID: BuDw70drzUY
f3a0002b4c71d7cc4839f0a4c63aebb6eda333ed: Bug 1419226 - Part 4. Fire DidPaint Event when revoke transaction. r?mattwoodrow draft
Mantaroh Yoshinaga <mantaroh@gmail.com> - Fri, 19 Jan 2018 16:54:29 +0900 - rev 722572
Push 96174 by bmo:mantaroh@gmail.com at Fri, 19 Jan 2018 07:56:06 +0000
Bug 1419226 - Part 4. Fire DidPaint Event when revoke transaction. r?mattwoodrow If the paint ended up making no change, gecko need to fire MozAfterPaint. This patch will send this event when revoke transaction. MozReview-Commit-ID: 8MUeFRq9GYE
2222e24e779b5d74da24771dfabc98689979b82a: Bug 1419226 - Part 3.Wait for MozAfterPaint instead of waiting for frames on file_deferred_start.html . r?hiro draft
Mantaroh Yoshinaga <mantaroh@gmail.com> - Fri, 19 Jan 2018 16:53:57 +0900 - rev 722571
Push 96174 by bmo:mantaroh@gmail.com at Fri, 19 Jan 2018 07:56:06 +0000
Bug 1419226 - Part 3.Wait for MozAfterPaint instead of waiting for frames on file_deferred_start.html . r?hiro This patch will : * Create test div element after waiting for document load, then wait for painting after it. * Change to waiting for MozAfterPaint event without workaround of waiting for excessive frames. MozReview-Commit-ID: 6Ytxln3tJi4
2aa96985175cf3aac0e367b0dc43ca1fc0426002: Bug 1419226 - Part 2. Remove MozAfterPaint timer and firing MozAfterPaint when revoke transaction. r?mattwoodrow draft
Mantaroh Yoshinaga <mantaroh@gmail.com> - Fri, 19 Jan 2018 16:53:27 +0900 - rev 722570
Push 96174 by bmo:mantaroh@gmail.com at Fri, 19 Jan 2018 07:56:06 +0000
Bug 1419226 - Part 2. Remove MozAfterPaint timer and firing MozAfterPaint when revoke transaction. r?mattwoodrow Current gecko will send MozAfterPaint event instead of finishing paint process due to software timer. This patch will remove this timer and ensure that firing MozAfterPaint when the paint ended up making no changes. MozReview-Commit-ID: E0ZNyeWHnrl
2804f95d9c2306c68b4214b2ec93ccd8f89df61f: Bug 1419226 - Part 1.Change observing target window of MozAfterPaint. r?mconley draft
Mantaroh Yoshinaga <mantaroh@gmail.com> - Fri, 19 Jan 2018 16:52:57 +0900 - rev 722569
Push 96174 by bmo:mantaroh@gmail.com at Fri, 19 Jan 2018 07:56:06 +0000
Bug 1419226 - Part 1.Change observing target window of MozAfterPaint. r?mconley Previous code, print preview waits for content window's MozAfterPaint. However gecko prevents send MozAfterPaint to content window[1]. So this code will not work correctly. However, software timer of firing MozAfterPaint ran this code.[2] This patch will * Change the observing content window to chrome window. * Add timer of MozAfterPaint event in order to ensure this event even if display list invalidation doesn't invalidate. Gecko create this timer in nsPresContext previously[2], but this bug will remove it [1] https://searchfox.org/mozilla-central/rev/919dce54f43356c22d6ff6b81c07ef412b1bf933/layout/base/nsPresContext.cpp#2452 [2] https://searchfox.org/mozilla-central/rev/919dce54f43356c22d6ff6b81c07ef412b1bf933/layout/base/nsPresContext.cpp#3466-3472 MozReview-Commit-ID: GcuKPjn0qhc
68b287a8b10a9814654d40d4c5c99b2279a2329f: Bug 1431632 - Update language name for locale "cak" to Kaqchikel r?gandalf draft
Francesco Lodolo (:flod) <flod@lodolo.net> - Fri, 19 Jan 2018 08:44:29 +0100 - rev 722568
Push 96173 by bmo:francesco.lodolo@gmail.com at Fri, 19 Jan 2018 07:53:20 +0000
Bug 1431632 - Update language name for locale "cak" to Kaqchikel r?gandalf MozReview-Commit-ID: 2f4ZS0xHW3t
ef2fc483c7342b25ef4912cfa55e1ade0877917e: o Bug 1431632 - Update language name for locale "cak" to Kaqchikel r?gandalf draft
Francesco Lodolo (:flod) <flod@lodolo.net> - Fri, 19 Jan 2018 08:44:29 +0100 - rev 722567
Push 96172 by bmo:francesco.lodolo@gmail.com at Fri, 19 Jan 2018 07:49:34 +0000
o Bug 1431632 - Update language name for locale "cak" to Kaqchikel r?gandalf MozReview-Commit-ID: 2f4ZS0xHW3t
6fac17d90c7a302fb78cdd7f7246d3c37505d653: Bug 1431632 - Update language name for locale "cak" to Kaqchikel r?pike draft
Francesco Lodolo (:flod) <flod@lodolo.net> - Fri, 19 Jan 2018 08:44:29 +0100 - rev 722566
Push 96171 by bmo:francesco.lodolo@gmail.com at Fri, 19 Jan 2018 07:48:17 +0000
Bug 1431632 - Update language name for locale "cak" to Kaqchikel r?pike MozReview-Commit-ID: 2f4ZS0xHW3t
cba108cfe14fae055dcb982a80f6ec5a3d0bef17: Bug 1430785 - Make mozInlineSpellChecker not derived from nsIEditActionListener r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 17 Jan 2018 14:05:06 +0900 - rev 722565
Push 96170 by masayuki@d-toybox.com at Fri, 19 Jan 2018 07:47:18 +0000
Bug 1430785 - Make mozInlineSpellChecker not derived from nsIEditActionListener r?m_kato mozInlineSpellChecker listens to only DidSplitNode() and DidJoinNodes(). So, EditorBase should call them directly rather than treating it as an nsIEditActionListener since its runtime cost becomes really cheaper. Different from EditorSpellCheck, nobody creates instance of nsIInlineSpellChecker. So, we can now stop supporting createInstance() for it in chrome JS and should do this since EditorBase cannot support multiple mozInlineSpellChecker instances without nsIEditActionListener interface. Therefore, this patch removes the entry from factory. MozReview-Commit-ID: W1CLdsJaaB
0d2fb26930cc562f243fce41ce477c1ad45b4249: Bug 1430982 - part 10: Make EditorBase store TextServicesDocument for its mInlineSpellChecker for avoiding to run for loops to call nsIEditActionListener methods r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 18 Jan 2018 23:46:03 +0900 - rev 722564
Push 96170 by masayuki@d-toybox.com at Fri, 19 Jan 2018 07:47:18 +0000
Bug 1430982 - part 10: Make EditorBase store TextServicesDocument for its mInlineSpellChecker for avoiding to run for loops to call nsIEditActionListener methods r?m_kato Currently, first edit action listener is always TextServicesDocument for EditorBase::mInlineSpellChecker if spell check is enabled and it requires only DidDeleteNode() and DidJoinNodes(). So, in most cases, EditorBase should call only them to avoid running redundant for loops for nsIEditActionListener methods. This patch makes that EditorBase::AddEditActionListener() and RemoveEditActionListener() check whether coming nsIEditActionListener is TextServicesDocument for mInlineSpellChecker. If so, EditorBase should store it as reference to TextServicesDocument. And when edit action occurs, its DidDeleteNode() and DidJoinNodes() should be called directly. Unfortunately, this patch makes TextServiceDocument's maintaining rules complicated. However, it must be really rare case to add new edit action listener because it's really old component. Note that EditorSpellCheck may be created multiple instances for an editor from chrome JS. Therefore, we need to keep TextServicesDocument being derived from nsIEditActionListener because in such case, TextServicesDocument for other EditorSpellCheck instances should be supported via nsIEditActionListener even though such case makes EditorBase slower. MozReview-Commit-ID: KtlXsBCOzKL
c3c35b29c2da88b224a68736096e3dd4ee246789: Bug 1430982 - part 9: Create accessors for each member of mozSpellChecker, EditorSpellCheck, mozInlineSpellChecker r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 18 Jan 2018 21:38:50 +0900 - rev 722563
Push 96170 by masayuki@d-toybox.com at Fri, 19 Jan 2018 07:47:18 +0000
Bug 1430982 - part 9: Create accessors for each member of mozSpellChecker, EditorSpellCheck, mozInlineSpellChecker r?m_kato mozSpellChecker stores TextServicesDocument, EditorSpellCheck stores mozSpellChecker and mozInlineSpellChecker stores EditorSpellCheck. So, they should have accessors for their member. Then, EditorBase can access all of them. MozReview-Commit-ID: Igphm8nRqve
4feb5378c777c3c5ead59e29db38234447f37aa4: Bug 1430982 - part 8: Make EditorBase store inline spell checker as mozInlineSpellChecker rather than nsIInlineSpellChecker r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 17 Jan 2018 00:52:37 +0900 - rev 722562
Push 96170 by masayuki@d-toybox.com at Fri, 19 Jan 2018 07:47:18 +0000
Bug 1430982 - part 8: Make EditorBase store inline spell checker as mozInlineSpellChecker rather than nsIInlineSpellChecker r?m_kato EditorBase::mInlineSpellChecker is always an instance of mozInlineSpellChecker. Fortunately, it's easy to expose mozInlineSpellChecker.h. So, making EditorBase store it as mozInlineSpellChecker directly, EditorBase can access any of mozInlineSpellChecker, EditorSpellCheck, mozSpellChecker and TextServicesDocument with new accessors created by following patch. MozReview-Commit-ID: 2oyS5tPeQcg
0bde4fa15152957aaf94ee7c95fd5a052c907b7c: Bug 1430982 - part 7: Make mozInlineSpellChecker store EditorSpellCheck directly rather than nsIEditorSpellCheck r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 18 Jan 2018 21:12:46 +0900 - rev 722561
Push 96170 by masayuki@d-toybox.com at Fri, 19 Jan 2018 07:47:18 +0000
Bug 1430982 - part 7: Make mozInlineSpellChecker store EditorSpellCheck directly rather than nsIEditorSpellCheck r?m_kato mozInlienSpellChecker::mSpellCheck and mozInlineSpellChecker::mPendingSpellCheck are always instances of EditorSpellCheck because mozInlienSpellChecker creates its instance. Therefore, making mozInlineSpellChecker store EditorSpellCheck directly makes sense. MozReview-Commit-ID: LTtEMZqMS9E
47c40ace261b5cf80173ebcb44ee41ced89145f6: Bug 1430982 - part 6: Rename nsEditorSpellCheck to mozilla::EditorSpellCheck and expose its header r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 18 Jan 2018 21:01:13 +0900 - rev 722560
Push 96170 by masayuki@d-toybox.com at Fri, 19 Jan 2018 07:47:18 +0000
Bug 1430982 - part 6: Rename nsEditorSpellCheck to mozilla::EditorSpellCheck and expose its header r?m_kato For making mozInlineSpellChecker stores nsEditorSpellCheck directly, we need to expose its header file. For doing that this patch renames the class to mozilla::EditorSpellCheck and expose it as "mozilla/EditorSpellCheck.h". MozReview-Commit-ID: 5H66Y2vVshu
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip