01bdca9df2c7f693ede320a0d5bd730921eb4907: Bug 1304922 - Part 6: Introduce mPropertiesWithImportantRules and mPropertiesForAnimationsLevel and use it to detect conditions that we need to update layers. r=birtles
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 05 Oct 2016 14:42:56 +0900 - rev 343274
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304922 - Part 6: Introduce mPropertiesWithImportantRules and mPropertiesForAnimationsLevel and use it to detect conditions that we need to update layers. r=birtles This patch also makes composite order lowest to highest, as a result we also need to replace mWinsInCascade checks with the the properties. The mWinsInCascade membed itself will be removed in a subsequent patch. Now we call RequestRestyle(Layer) respectively for transition and animation, so a test case in test_restyles.html works as expected. And now lower-priority animations are also sent to the compositor so this patch fixed some tests in test_running_on_compositor.html and test_animation_performance_warning.html MozReview-Commit-ID: BchUsJbmatg
04b7daffe4383dfa772201b4d2e0281409ea6e16: Bug 1304922 - Part 5: AnimValuesStyleRule::AddValue replaces the existence entry's mValue. r=birtles
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 05 Oct 2016 14:26:44 +0900 - rev 343273
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304922 - Part 5: AnimValuesStyleRule::AddValue replaces the existence entry's mValue. r=birtles MozReview-Commit-ID: GqpUsXUZaHy
37ffd2ddd1ec9be1a5d4a51e8681ee8932684550: Bug 1304922 - Part 4: Add EffectSet::Count(). r=birtles
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 05 Oct 2016 14:26:32 +0900 - rev 343272
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304922 - Part 4: Add EffectSet::Count(). r=birtles MozReview-Commit-ID: HqXNf4938Rb
0c0cf80887d35b5b551084ed05e17420ef2741ee: Bug 1304922 - Part 3: Request restyle for layer when CSS animation's index is changed. r=birtles
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 05 Oct 2016 14:26:29 +0900 - rev 343271
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304922 - Part 3: Request restyle for layer when CSS animation's index is changed. r=birtles In the case when the animation index is changed, we have to update the layer because the index affects composite order. Currently UpdateCascadeResult() does it but the function will check only which cascade level wins in the cascade level and request restyle only when the winner is changed, e.g. when a CSS animation overrides a CSS transition, etc. So we should call request restyle respectively when the index is changed inside each Animation class. MozReview-Commit-ID: KcEFyBpALDA
df062c00654d9e96407c7704bff9c4de5674d595: Bug 1304922 - Part 2: Sort animation array before sending animations to the compositor. r=birtles
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 05 Oct 2016 14:26:25 +0900 - rev 343270
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304922 - Part 2: Sort animation array before sending animations to the compositor. r=birtles The result animatons of FindAnimationsForCompositor() should be sorted by composite order because SampleAnimations() on the compositor expects the animations are sorted by the order. MozReview-Commit-ID: 4eI4EldFVcu
8b74bede29c3ea65bbeb6561d34501b156e59662: Bug 1304922 - Part 1: Rename nsLayoutUtils::HasCurrentAnimationOfProperty() to nsLayoutUtils::HasActiveAnimationOfProperty(). r=birtles
Hiroyuki Ikezoe <hiikezoe@mozilla-japan.org> - Wed, 05 Oct 2016 14:25:41 +0900 - rev 343269
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304922 - Part 1: Rename nsLayoutUtils::HasCurrentAnimationOfProperty() to nsLayoutUtils::HasActiveAnimationOfProperty(). r=birtles KeyframeEffectReadOnly::HasAnimationOfProperty() calls GetAnimationOfProperty() which checks mWinsInCascade flag and the mWinsInCascade flag is set to true only if the effect is in-effect. That means nsLayoutUtils::HasCurrentAnimationOfProperty() actually represents that a given frame has at least one animation which is current and *in-effect* (i.e. active). MozReview-Commit-ID: 93rMMmzrBMi
c7d62e6d052c5d2638b08d480a720254ea09ff2d: Backed out changeset ad324639bfae (bug 1304178) for slowing down debugger tests enough to make their Linux32 debug test chunks exceed the 80 minute limit
Phil Ringnalda <philringnalda@gmail.com> - Wed, 05 Oct 2016 20:42:31 -0700 - rev 343268
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Backed out changeset ad324639bfae (bug 1304178) for slowing down debugger tests enough to make their Linux32 debug test chunks exceed the 80 minute limit MozReview-Commit-ID: L8rkkQxTdff
e8fa13708c070d1fadf488ed9d951464745b4e17: merge mozilla-inbound to mozilla-central a=merge
Carsten "Tomcat" Book <cbook@mozilla.com> - Wed, 05 Oct 2016 15:42:48 +0200 - rev 343267
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
merge mozilla-inbound to mozilla-central a=merge
7cedca0e724800f47dc7d0b3ed31683f18829537: Bug 1306920 - Make sure log file is closed when clearing it at runtime r=nfroyd
Valentin Gosu <valentin.gosu@gmail.com> - Wed, 05 Oct 2016 11:34:12 +0200 - rev 343266
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1306920 - Make sure log file is closed when clearing it at runtime r=nfroyd
af72e3cf941f708a58ec68d1a4ba5dfe669e35c0: Bug 1283563 - Followup - properly finish test in case timeout is really flaky. r=me
Kan-Ru Chen <kanru@kanru.info> - Wed, 05 Oct 2016 16:31:51 +0800 - rev 343265
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1283563 - Followup - properly finish test in case timeout is really flaky. r=me MozReview-Commit-ID: 9XkB9dExF7Y
0c8c3b25f9a489b8803407aaf763097ba69d5a34: Bug 1307372 - Avoid triggering mCallback read barrier in CallbackObject::CallbackKnownNotGray r=mccr8
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 05 Oct 2016 10:09:24 +0200 - rev 343264
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1307372 - Avoid triggering mCallback read barrier in CallbackObject::CallbackKnownNotGray r=mccr8
884149c8dbc24e11f48f638c0ad94205b3719f13: Bug 1307361 - Remove OOM message in heap checking zeal mode r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 05 Oct 2016 10:09:24 +0200 - rev 343263
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1307361 - Remove OOM message in heap checking zeal mode r=sfink
e7bb6f6235cd9c3d5377f425112175fc55ac7d99: Bug 1302682 - Add assertion to help catch incorrect alloc kind when tenuring r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 05 Oct 2016 10:09:24 +0200 - rev 343262
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1302682 - Add assertion to help catch incorrect alloc kind when tenuring r=sfink
2e699b292594252b7f2588def0a88418d81377e3: Bug 1304975 - Use the renamed gUM permission token. r=florian
Gian-Carlo Pascutto <gcp@mozilla.com> - Fri, 23 Sep 2016 12:31:53 +0200 - rev 343261
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304975 - Use the renamed gUM permission token. r=florian MozReview-Commit-ID: 5pJorbXrtkP
54600a72f41a84814bdb8dcb61798d1c43f86e1b: Bug 1307375: Add the use of MOZ_LOG to windows sandbox logging. r=jimm
Bob Owen <bobowencode@gmail.com> - Wed, 05 Oct 2016 08:10:46 +0100 - rev 343260
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1307375: Add the use of MOZ_LOG to windows sandbox logging. r=jimm
f88ef7f3c2d57d9e200e945e627c2817793d139c: Bug 1304441 Part 3 - Insert main summary's frame construction item at front of the list. r=bz
Ting-Yu Lin <tlin@mozilla.com> - Wed, 05 Oct 2016 14:43:32 +0800 - rev 343259
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304441 Part 3 - Insert main summary's frame construction item at front of the list. r=bz Change the logic that moves the main summary to the front from operating on generated frames in DetailsFrame::SetInitialChildList() to operating on frame construction item list in AddFrameConstructionItemsInternal() so that it will be correct when cooperating with ::first-line. The root cause of the bug reported is because when specifying ::first-line on details element, the first frame of summary element, which is generated due to ib-split, will be wrapped in nsFirstLineFrame. The original code fails to find the summary frame in the wrapper frame and triggers assertion because of the second ib-split summary frame. To fix that, we need to descend into the child list of wrapper frames when checking the main summary. Add original test case as a crashtest as well as reftests to clearly reproduce the issue. Note that in the reftest, the blue color in ::first-line is applied incorrectly to the second line in the summary due to bug 520605. MozReview-Commit-ID: Bv4Vcvxp6pY
9f5724ff0f7ce4d03a36ab91a6c802ae3c73aa76: Bug 1304441 Part 2 - Extract main summary checking code to a function. r=bz
Ting-Yu Lin <tlin@mozilla.com> - Wed, 05 Oct 2016 14:43:31 +0800 - rev 343258
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304441 Part 2 - Extract main summary checking code to a function. r=bz * Change the assertion to non-fatal to make it easier to debug. * Change the wording per bug 1304441 comment 11. MozReview-Commit-ID: 1UJXhC4qkrx
f41d60821e3c3b7bb0ee86c524e5ffdcdde5fdef: Bug 1304441 Part 1 - Remove unneeded check when constructing frame for summary. r=bz
Ting-Yu Lin <tlin@mozilla.com> - Wed, 05 Oct 2016 14:43:31 +0800 - rev 343257
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1304441 Part 1 - Remove unneeded check when constructing frame for summary. r=bz SummaryFrame had been removed in bug 1258657, so now HTMLSummaryElement is always rendered as an ordinary inline or block frame. Therefore, the check in FindHTMLData is not needed anymore. MozReview-Commit-ID: Ikxla6QoNLT
032113ce311c9d37c5fc0d9c9e14f24ba2cfc63a: Bug 1283563 - Change geo.wifi.url BASE_URL based on test type. r=jdm
Kan-Ru Chen <kanru@kanru.info> - Mon, 03 Oct 2016 17:48:52 +0800 - rev 343256
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1283563 - Change geo.wifi.url BASE_URL based on test type. r=jdm MozReview-Commit-ID: CbyHe017B25
a3d11cbf8bfde73525f8c4795391a32548697d10: Bug 1283563 - Add mochitest/geolocation/chrome.ini back. r=jdm
Kan-Ru Chen <kanru@kanru.info> - Mon, 03 Oct 2016 17:58:14 +0800 - rev 343255
Push 10298 by raliiev@mozilla.com at Mon, 14 Nov 2016 12:33:03 +0000
Bug 1283563 - Add mochitest/geolocation/chrome.ini back. r=jdm MozReview-Commit-ID: IH7RkPaumCk
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip