e3741d5b60f182cddec58733e2d17cdf4dfe586b: Bug 1308876 - Mark lines dirty when we abort their reflow due to page-break-inside:avoid. r=mats
L. David Baron <dbaron@dbaron.org> - Wed, 12 Jul 2017 19:37:11 -0700 - rev 608039
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 1308876 - Mark lines dirty when we abort their reflow due to page-break-inside:avoid. r=mats Both of the changes are needed to fix layout/reftests/w3c-css/submitted/css21/pagination/moz-css21-float-page-break-inside-avoid-4.html with the primary patch in bug 1308876. That patch changes the transfer of NS_FRAME_IS_DIRTY from parent to child so that it happens at the start of the parent's reflow rather than later at the start of the child's reflow, which means that frames that are pulled into a dirty frame during reflow are not marked dirty (and thus forced to reflow all of their lines). This means that the primary patch in bug 1308876 introduces the possibility of non-dirty reflows during printing, which means we exercise non-dirty relayout code in a number of tests where we did not do so previously. Note: This patch passes try on its own, on Linux64 debug. Writing a simple test for this that fails without the primary patch in bug 1308876 seems difficult. ColumnSetFrame responds to nsReflowStatus::NextInFlowNeedsReflow by marking the next-in-flow *dirty* (which page frames don't), which makes it hard to test in columns, at least without nesting. (Colums probably shouldn't do that, though, but that's a performance fix for another time.) MozReview-Commit-ID: JZ3qWTSO2lX
f8c5d7e8c28ad64e495e0534d40975d8ab736772: Bug 1380521 - Use a more reliably invalid initial mBSizeOfARow so that we reflow properly if it changes to zero. r=dholbert
L. David Baron <dbaron@dbaron.org> - Wed, 12 Jul 2017 19:37:11 -0700 - rev 608038
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 1380521 - Use a more reliably invalid initial mBSizeOfARow so that we reflow properly if it changes to zero. r=dholbert This fixes a failure in layout/reftests/bugs/467084-1.html, triggered by the primary patch in bug 1308876, on Android only (or on all platforms if I add 'html { overflow: hidden }'). I confirmed locally (on Linux) that, without this patch, both layout/reftests/bugs/467084-1.html with 'html { overflow: hidden }', and the reftest added here, fail, but that with the patch they both pass. MozReview-Commit-ID: 3SjTlnBngBV
4bf16638687ffa908a78d7ab2d04320c36afcaba: Bug 1380521 - Fix spelling of mSuppressScrollbarUpdate member variable. r=dholbert
L. David Baron <dbaron@dbaron.org> - Wed, 12 Jul 2017 19:37:11 -0700 - rev 608037
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 1380521 - Fix spelling of mSuppressScrollbarUpdate member variable. r=dholbert MozReview-Commit-ID: 6i2cz2Flwtb
5631a4402393f2b4f16b4f959436eea1e717fe43: Bug 1380521 - Remove pointless assertion. r=dholbert
L. David Baron <dbaron@dbaron.org> - Wed, 12 Jul 2017 19:37:11 -0700 - rev 608036
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 1380521 - Remove pointless assertion. r=dholbert It's just asserting the same as the test that contains it. MozReview-Commit-ID: KS58GuvVZDA
905f598f2051286c68f860cca497e75fb8cdeaa0: Bug 1379334 - Make XULScrollFrame test for needing a vertical scrollbar because of the size of the horizontal scrollbar. r=dholbert
L. David Baron <dbaron@dbaron.org> - Wed, 12 Jul 2017 19:37:11 -0700 - rev 608035
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 1379334 - Make XULScrollFrame test for needing a vertical scrollbar because of the size of the horizontal scrollbar. r=dholbert This fixes an incremental layout bug, where the number of times we reflow the frame affects its layout. This is because we make the decisions about the vertical scrollbar before the horizontal scrollbar (and, when making the decision, adjust mHelper.mScrollPort for the size of the scrollbar). Thus, in order to avoid a situation where reflowing the scrollframe once leads us to have no vertical scrollbar, but reflowing it a second time leads us to add that scrollbar, we need to retest for the need for a vertical scrollbar after we add the horizontal one. (It's possible there are some other missing cases here, but this is the one that (a) already existed in the code and (b) is needed to fix the reftest failure on Windows that I got on bug 1308876, in layout/reftests/text-overflow/xulscroll.html . The reftest here shows the bug even without bug 1308876 (though I confirmed that only by loading the test and reference in a nightly build, not in the reftest harness). I did test that, in combination with bug 1308876, the test fails without the patch and passes with the patch. MozReview-Commit-ID: LhMi7LbmB6J
f676d48fd757855c0e0e52d0a3bde29c3ae4df2f: Bug 1379334 - Convert mis-indented code to 2-space indent, plus bracing and logical operator position fixes when reindenting. r=dholbert
L. David Baron <dbaron@dbaron.org> - Wed, 12 Jul 2017 19:37:11 -0700 - rev 608034
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 1379334 - Convert mis-indented code to 2-space indent, plus bracing and logical operator position fixes when reindenting. r=dholbert MozReview-Commit-ID: ElsSNF40LZQ
48f19f8bc8ba126fd28b8f3e90bd25610814fc09: Bug 667079. Make sure to set the right containing block size for inner tables no matter what. r=dbaron
Boris Zbarsky <bzbarsky@mit.edu> - Sat, 25 Jun 2011 01:21:47 -0400 - rev 608033
Push 68169 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 05:25:33 +0000
Bug 667079. Make sure to set the right containing block size for inner tables no matter what. r=dbaron MozReview-Commit-ID: 3EwrNfgJ2oA
ba9597a0307a71ac9a6a58f259ff106e4f8a60bc: Bug 1380567 - Remove RestyleManagerHandle. draft
Ting-Yu Lin <tlin@mozilla.com> - Thu, 13 Jul 2017 13:16:55 +0800 - rev 608032
Push 68168 by bmo:tlin@mozilla.com at Thu, 13 Jul 2017 05:17:27 +0000
Bug 1380567 - Remove RestyleManagerHandle. MozReview-Commit-ID: ArWT8FpvrCK
d60efaec60ed0a239f92d00c1cb509f4cb580a05: Bug 1365133 - Update tests - Part 2. draft
Evan Tseng <evan@tseng.io> - Tue, 20 Jun 2017 15:11:46 +0800 - rev 608031
Push 68167 by bmo:evan@tseng.io at Thu, 13 Jul 2017 05:02:09 +0000
Bug 1365133 - Update tests - Part 2. MozReview-Commit-ID: GoOXdJ4RfY9
385204b399d61d68c62b489b88a436a51b645b9e: Bug 1380556 - Darken the font color of also fill note on the dropdown footer. r=lchang draft
Ray Lin <ralin@mozilla.com> - Thu, 13 Jul 2017 12:54:29 +0800 - rev 608030
Push 68166 by bmo:ralin@mozilla.com at Thu, 13 Jul 2017 04:55:45 +0000
Bug 1380556 - Darken the font color of also fill note on the dropdown footer. r=lchang MozReview-Commit-ID: 1QbW7zPXs10
d855cff0792741eceb6a3977752247d9fdd8e3ef: Bug 1380301 - Add windows.h inculde to nsLookAndFeel.h draft
Farmer Tseng <fatseng@mozilla.com> - Thu, 13 Jul 2017 12:46:26 +0800 - rev 608029
Push 68165 by bmo:fatseng@mozilla.com at Thu, 13 Jul 2017 04:46:46 +0000
Bug 1380301 - Add windows.h inculde to nsLookAndFeel.h MozReview-Commit-ID: Js9Kc73VXS1
f24833d8a06b660462b8ac7583425dfb82da5581: Bug 1380301 - Add missing "#ifdef MOZ_ENABLE_SKIA_PDF" to nsDeviceContextSpecWin.h draft
Farmer Tseng <fatseng@mozilla.com> - Thu, 13 Jul 2017 12:46:21 +0800 - rev 608028
Push 68165 by bmo:fatseng@mozilla.com at Thu, 13 Jul 2017 04:46:46 +0000
Bug 1380301 - Add missing "#ifdef MOZ_ENABLE_SKIA_PDF" to nsDeviceContextSpecWin.h MozReview-Commit-ID: IHxE0SiAlzg
150bc51e6d437077da2fc497e307d53d08ca5655: Bug 1380301 - Add windows.h inculde to nsLookAndFeel.h draft
Farmer Tseng <fatseng@mozilla.com> - Thu, 13 Jul 2017 12:08:04 +0800 - rev 608027
Push 68164 by bmo:fatseng@mozilla.com at Thu, 13 Jul 2017 04:44:33 +0000
Bug 1380301 - Add windows.h inculde to nsLookAndFeel.h MozReview-Commit-ID: Js9Kc73VXS1
250b0c2cbe650ba1f0a09c0ef6a6b165a464cdb0: Bug 1380301 - Add missing "#ifdef MOZ_ENABLE_SKIA_PDF" to nsDeviceContextSpecWin.h draft
Farmer Tseng <fatseng@mozilla.com> - Thu, 13 Jul 2017 12:01:46 +0800 - rev 608026
Push 68164 by bmo:fatseng@mozilla.com at Thu, 13 Jul 2017 04:44:33 +0000
Bug 1380301 - Add missing "#ifdef MOZ_ENABLE_SKIA_PDF" to nsDeviceContextSpecWin.h MozReview-Commit-ID: IqPe5isRufF
a5ea943585644f6dd295289648f50ce34ca60df7: Bug 1369942 - Display Negative Line Numbers in CSS Grid Inspector. r?gl draft
Micah Tigley <tigleym@gmail.com> - Wed, 12 Jul 2017 21:11:30 -0600 - rev 608025
Push 68163 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 04:33:48 +0000
Bug 1369942 - Display Negative Line Numbers in CSS Grid Inspector. r?gl MozReview-Commit-ID: IbijTE5TJW5
e6e712904806da25a9c8f48ea4533abe7c6ea8f4: Bug 1379628 - Ensure pixels for Android dynamic toolbar snapshot get processed even if the toolbar is not visible so pending animations may run r=kats
Randall Barker <rbarker@mozilla.com> - Wed, 12 Jul 2017 12:54:59 -0700 - rev 608024
Push 68163 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 04:33:48 +0000
Bug 1379628 - Ensure pixels for Android dynamic toolbar snapshot get processed even if the toolbar is not visible so pending animations may run r=kats MozReview-Commit-ID: 9iMFN2gdfKl
1023371e0802e3a22cfac58f1f7f3968a6112cfe: Merge m-c to inbound, a=merge
Wes Kocher <wkocher@mozilla.com> - Wed, 12 Jul 2017 18:26:46 -0700 - rev 608023
Push 68163 by bmo:tigleym@gmail.com at Thu, 13 Jul 2017 04:33:48 +0000
Merge m-c to inbound, a=merge MozReview-Commit-ID: 7rfXDtMWy2C
ec4d837145c42db25853dfbf2b3553fc9c4e3039: Bug 1378691 P4 - skip to next key frame if the playback position is very close to it; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 12 Jul 2017 17:03:45 +0800 - rev 608022
Push 68162 by bmo:kaku@mozilla.com at Thu, 13 Jul 2017 04:21:49 +0000
Bug 1378691 P4 - skip to next key frame if the playback position is very close to it; r?jwwang MozReview-Commit-ID: Gm0iZAHVLpS
82130dcce51d6f47502e0fa7adbd056d803a0008: Bug 1378691 P3 - return next key frame time back to MDSM; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Tue, 11 Jul 2017 10:15:33 +0800 - rev 608021
Push 68162 by bmo:kaku@mozilla.com at Thu, 13 Jul 2017 04:21:49 +0000
Bug 1378691 P3 - return next key frame time back to MDSM; r?jwwang MozReview-Commit-ID: LVhJ0DpM24c
a6392a6b15c9a4ecfbda6a5daee5f49d205dff53: Bug 1378691 P2 - trigger skip-to-next-key-frame if the video decoding is too slow; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Tue, 11 Jul 2017 09:59:13 +0800 - rev 608020
Push 68162 by bmo:kaku@mozilla.com at Thu, 13 Jul 2017 04:21:49 +0000
Bug 1378691 P2 - trigger skip-to-next-key-frame if the video decoding is too slow; r?jwwang MozReview-Commit-ID: B0stJG3eOKU
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip