26f5729d5ccc245abe201fb339b6b16351c148b4: Bug 1120252 - Avoid trying to get the APZCTreeManager if APZ isn't enabled. r=mattwoodrow
Kartikaya Gupta <kgupta@mozilla.com> - Sun, 11 Jan 2015 20:39:36 -0500 - rev 223176
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1120252 - Avoid trying to get the APZCTreeManager if APZ isn't enabled. r=mattwoodrow
ebb112ced42c29f85fdd5947d25c921c24441ddd: Bug 1119117. Don't clip nsDisplayOpacity items, since some of its descendants maybe should not be clipped with the nsDisplayOpacity clip. r=mattwoodrow
Robert O'Callahan <robert@ocallahan.org> - Sun, 11 Jan 2015 00:12:33 +1300 - rev 223175
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1119117. Don't clip nsDisplayOpacity items, since some of its descendants maybe should not be clipped with the nsDisplayOpacity clip. r=mattwoodrow
98e34be1fb44344c1173d9d3c981b3e5a2da4762: Revert one of the assertion additions in Bug 1110277 patch 3 since it fires, causing orange. r=orange
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 16:58:42 -0800 - rev 223174
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Revert one of the assertion additions in Bug 1110277 patch 3 since it fires, causing orange. r=orange Example stack, from editor/libeditor/tests/browserscope/test_richtext2.html : ###!!! ASSERTION: should only call on first continuation/ib-sibling: 'nsLayoutUtils::IsFirstContinuationOrIBSplitSibling(this)', file /builds/slave/m-in-l64-d-0000000000000000000/build/src/layout/base/../generic/nsIFrame.h, line 875 #01: AdjustAppendParentForAfterContent [layout/base/nsCSSFrameConstructor.cpp:6059] #02: nsCSSFrameConstructor::ContentAppended(nsIContent*, nsIContent*, bool) [layout/base/nsCSSFrameConstructor.cpp:7155] #03: PresShell::ContentAppended(nsIDocument*, nsIContent*, nsIContent*, int) [layout/base/nsPresShell.cpp:4520] #04: nsNodeUtils::ContentAppended(nsIContent*, nsIContent*, int) [dom/base/nsNodeUtils.cpp:132] #05: nsINode::doInsertChildAt(nsIContent*, unsigned int, bool, nsAttrAndChildArray&) [dom/base/nsINode.cpp:1544] #06: nsINode::ReplaceOrInsertBefore(bool, nsINode*, nsINode*, mozilla::ErrorResult&) [dom/base/nsINode.cpp:2209] #07: mozilla::dom::NodeBinding::appendChild [obj-firefox/dom/bindings/NodeBinding.cpp:600]
1bbebe9fec17e88234845d8da22c8b44f394121b: Bug 1110277 patch 3 - Look for the GenConPseudos() property on the first continuation/ib-split so that we can find it when looking for the ::after frame. r=bzbarsky
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 15:43:11 -0800 - rev 223173
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1110277 patch 3 - Look for the GenConPseudos() property on the first continuation/ib-split so that we can find it when looking for the ::after frame. r=bzbarsky The change to GetAfterFrameForContent prevents the reframe that is part of the chain of events leading to this bug, and thus fixes the bug on its own. The change to GetBeforeFrameForContent seems desirable for symmetry. Note that patch 6 also independently fixes the reported bug. This probably needs somewhat careful review. We should examine: (1) what the rules for calling nsLayoutUtils::GetBeforeFrame and nsLayoutUtils::GetAfterFrame are, and whether both (or neither) need to be patched. (2) What the rules are for which frame the GenConProperty() lives on, and whether we should adjust nsIFrame::GetGenConPseudos() to either do something more intelligent, or assert about callers. (We should probably clean up some of these things in a followup bug.) Since the symptom of this bug is (once patch 4 is in the tree) only causing extra reframes, it can only be tested using the new API (from bug 1115691) for observing reframes. I confirmed that the test for this bug fails without the patch and passes with the patch (as noted by the removal of its todo annotation). This patch fixes the assertion on layout/generic/crashtests/600100.xhtml, though I haven't investigated why.
fbbafc2a957318cc4fa2c2dbfd774124ccf597be: Bug 1110277 patch 2 - Add nsLayoutUtils::LastContinuationOrIBSplitSibling(). r=bzbarsky
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 15:43:10 -0800 - rev 223172
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1110277 patch 2 - Add nsLayoutUtils::LastContinuationOrIBSplitSibling(). r=bzbarsky This is used in patch 3.
86088b6c69e8414e6a9845fc7985de6dbbf6f16e: Bug 1110277 patch 1 - Consider the ib-split chain when checking for ::before and ::after in order to reframe when they're missing. r=bzbarsky
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 15:43:10 -0800 - rev 223171
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1110277 patch 1 - Consider the ib-split chain when checking for ::before and ::after in order to reframe when they're missing. r=bzbarsky This patch is not needed to fix the bug, but it seems like it's probably desirable. It's not needed for this bug because MaybeReframeForBeforePseudo and MaybeReframeForAfterPseudo are already called (by ElementRestyler::RestyleChildren) on only the first and last continuation or ib-split sibling with the same style. So this patch should only actually change anything for cases like a block-in-inline split whose initial inline part is inside of a ::first-line (where different parts of the block-in-inline split chain have different style). Since the symptom of this bug is (once patch 6 is in the tree) only causing extra reframes, it can only be tested using the new API (from bug 1115691) for observing reframes. I confirmed that the test for this bug fails without the patch and passes with the patch (as noted by the removal of its todo annotation).
ca02a661d6dc7a4a844488e30961da354ae7227c: Bug 1110277 patch 0 - Add mochitest for patch 1 and patch 3. r=heycam
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 15:43:10 -0800 - rev 223170
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1110277 patch 0 - Add mochitest for patch 1 and patch 3. r=heycam Patch 1 and patch 3 will change the todo status of the first and second tests, respectively, since they are what fix the tests.
9fe58932c19292f5df92507aa611db1506ea51d5: Bug 1115691 patch 2 - Expose counts of frames constructed and reflowed on DOMWindowUtils. r=roc
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 15:43:10 -0800 - rev 223169
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1115691 patch 2 - Expose counts of frames constructed and reflowed on DOMWindowUtils. r=roc This counts are intended to be useful for: * tests * perhaps developer tools that need to determine whether frame construction or reflow has happened in a particular interval.
d1510c5deff9c9972fda8edfebaac493d3ea5635: Bug 1115691 patch 1 - Store counts of frames constructed and reflowed on the pres context. r=roc
L. David Baron <dbaron@dbaron.org> - Sun, 11 Jan 2015 15:43:10 -0800 - rev 223168
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1115691 patch 1 - Store counts of frames constructed and reflowed on the pres context. r=roc (I've tested the frame construction counts in my test for bug 1110277; I haven't actually tested the reflow counts.)
5c772b35c40dd6d6d3966f7f3f02c2d9032acaae: Bug 1120211 - Don't force the innerwindow size to the outerwindow size if the viewport is overridden. r=dbaron
Kartikaya Gupta <kgupta@mozilla.com> - Sun, 11 Jan 2015 18:26:39 -0500 - rev 223167
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1120211 - Don't force the innerwindow size to the outerwindow size if the viewport is overridden. r=dbaron In Fennec, if the root window gets resized, background tabs get a delayed resize event queued up. When those background tabs get painted next, the delayed resize is flushed. However, the code that flushes the delayed resize does not check to see if the viewport is already overridden. This is in contrast to all the other code paths that propagate outerwindow size change to the innerwindow, and is incorrect because it effectively clobbers any meta-viewport tag with the outerwindow dimensions. This patch corrects this code path to also check if the viewport is overridden.
010693d8f889809b9415019f0beb0845bc52b6f7: Bug 1120214 - Fix logging-only build failure. r=botond
Kartikaya Gupta <kgupta@mozilla.com> - Sun, 11 Jan 2015 18:26:26 -0500 - rev 223166
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1120214 - Fix logging-only build failure. r=botond
93c6ac70dc60ec1442c97ee4d22648c3551286fe: Bug 1119217 - Implement %TypedArray%.prototype.{keys, values, entries}. r=till
Tom Schuster <evilpies@gmail.com> - Sun, 11 Jan 2015 21:21:35 +0100 - rev 223165
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1119217 - Implement %TypedArray%.prototype.{keys, values, entries}. r=till
86f23a66df4a6c5bccac46ffeb1be7801101af61: Bug 1113062 - IndexedDB FileSnapshot not CCed, r=janv
Andrea Marchesini <amarchesini@mozilla.com> - Sun, 11 Jan 2015 21:35:24 +0000 - rev 223164
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1113062 - IndexedDB FileSnapshot not CCed, r=janv
d6581084a01f38ebaf28a4629d6e69e95071a03f: Bug 1113062 - patch 2 - ArchiveReaderZipFile non-CCed, r=smaug
Andrea Marchesini <amarchesini@mozilla.com> - Sun, 11 Jan 2015 21:34:57 +0000 - rev 223163
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1113062 - patch 2 - ArchiveReaderZipFile non-CCed, r=smaug
a0ec121ce6bffb7aff20ae78be3fa7136ec0560b: Bug 1113062 - patch 1 - PIFileImpl and FileImpl merged, r=smaug
Andrea Marchesini <amarchesini@mozilla.com> - Sun, 11 Jan 2015 21:34:31 +0000 - rev 223162
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1113062 - patch 1 - PIFileImpl and FileImpl merged, r=smaug
3ffd66a3f1bd9d2bca1acf5c6086c85dec1119c2: Bug 1119456 - Work around the fact that media cache does not quite guarantee the property we want. r=roc
Bobby Holley <bobbyholley@gmail.com> - Sun, 11 Jan 2015 13:24:26 -0800 - rev 223161
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1119456 - Work around the fact that media cache does not quite guarantee the property we want. r=roc
19b8c420b13a31d1d1e12cc642493be2e82059de: Bug 1119456 - Make MP4Demuxer's blocking reads non-blocking and hoist blocking into callers with a hacky retry strategy. r=k17e
Bobby Holley <bobbyholley@gmail.com> - Sun, 11 Jan 2015 13:24:26 -0800 - rev 223160
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1119456 - Make MP4Demuxer's blocking reads non-blocking and hoist blocking into callers with a hacky retry strategy. r=k17e
7e4003516dd5f56463945c8e06515f28abefa83c: Bug 1118528 - Increase fuzz threshold for mp4. r=ajones
Matt Woodrow <mwoodrow@mozilla.com> - Mon, 12 Jan 2015 09:48:23 +1300 - rev 223159
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1118528 - Increase fuzz threshold for mp4. r=ajones
6f70785ed447f102427cc01a76cdb1ddfe14fce6: Bug 1118528 - Use fuzzy comparisons in MediaSourceReader::HaveData. r=cajbir
Matt Woodrow <mwoodrow@mozilla.com> - Mon, 12 Jan 2015 09:47:56 +1300 - rev 223158
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1118528 - Use fuzzy comparisons in MediaSourceReader::HaveData. r=cajbir
c9177ec839af085abfe9390c947e1a277a3a5e80: Bug 1116284 - Don't run MP4Reader::Update after we've shut the reader down. r=cpearce
Matt Woodrow <mwoodrow@mozilla.com> - Mon, 12 Jan 2015 09:43:11 +1300 - rev 223157
Push 28082 by cbook@mozilla.com at Mon, 12 Jan 2015 10:44:52 +0000
Bug 1116284 - Don't run MP4Reader::Update after we've shut the reader down. r=cpearce
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip