33824395a6ca9145c8b8ce474f043081181164d3: Bug 1317937: Rename nsImageMap::mContainsBlockContents to mConsiderWholeSubtree so it is more descriptive of what it actually does. r=mats
Timothy Nikkel <tnikkel@gmail.com> - Wed, 08 Nov 2017 16:02:05 +0100 - rev 441729
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1317937: Rename nsImageMap::mContainsBlockContents to mConsiderWholeSubtree so it is more descriptive of what it actually does. r=mats MozReview-Commit-ID: 4TpkkWmMIrc
8e5b21f2547f7ad3f4fed61e7af2e3c905d13830: Bug 1374523 - [Form Autofill] Increase timer to wait for dropdown rendering and run keyboard test first in browser_autocomplete_footer.js. r=ralin
Luke Chang <lchang@mozilla.com> - Thu, 09 Nov 2017 16:29:35 +0800 - rev 441728
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1374523 - [Form Autofill] Increase timer to wait for dropdown rendering and run keyboard test first in browser_autocomplete_footer.js. r=ralin MozReview-Commit-ID: EvUZuMx0PPA
ffd79ce93abd8199f4c98540b9cd32c8dc4dfe38: Bug 1414981 - Request URL should not show #hash;r=Honza
Fred Lin <gasolin@gmail.com> - Tue, 07 Nov 2017 13:46:57 +0800 - rev 441727
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1414981 - Request URL should not show #hash;r=Honza MozReview-Commit-ID: G3FqltGcaTC
798de42346d14e277a4f2de2189d4e1870eeef8e: Bug 1415345 - Fix undefined variable in error message in FxAccountsStorageManager#getAccountData. r=markh
Mark Banner <standard8@mozilla.com> - Tue, 07 Nov 2017 23:00:26 +0000 - rev 441726
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415345 - Fix undefined variable in error message in FxAccountsStorageManager#getAccountData. r=markh MozReview-Commit-ID: 4mCdPfKX5lz
2f059a9dbd9d0b76230f53a78bf29dfad1e56d06: Bug 1415069 - Add a member to ChannelMediaResource to remember seekability of the channel. r=bechen,gerald
JW Wang <jwwang@mozilla.com> - Tue, 07 Nov 2017 14:26:10 +0800 - rev 441725
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415069 - Add a member to ChannelMediaResource to remember seekability of the channel. r=bechen,gerald So it doesn't need to call mCacheStream.IsTransportSeekable() which needs to take the lock and might block the main thread. MozReview-Commit-ID: 99QVcSxzjCz
b12b1f6beb496de17cfcaa8af70d7216853b2866: Bug 1415766. P2 - move Seek() to private and tighten up some assertions. r=bechen,gerald
JW Wang <jwwang@mozilla.com> - Thu, 02 Nov 2017 14:46:24 +0800 - rev 441724
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415766. P2 - move Seek() to private and tighten up some assertions. r=bechen,gerald MozReview-Commit-ID: BBsXqKUrOi1
753f3591930d7a1ef1cd2a8e4cbe9e0f6b57d901: Bug 1415766. P1 - we never pass anything other than NS_SEEK_SET to Seek(). r=bechen,gerald
JW Wang <jwwang@mozilla.com> - Thu, 02 Nov 2017 14:26:24 +0800 - rev 441723
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415766. P1 - we never pass anything other than NS_SEEK_SET to Seek(). r=bechen,gerald MozReview-Commit-ID: AgvapCwwSpr
05002ad2b8f98d392a10811ce12f332ce2258210: Bug 1415397 - use Span<> to replace low level pointer arithmetic in ReadFromCache(). r=bechen,gerald
JW Wang <jwwang@mozilla.com> - Thu, 02 Nov 2017 11:36:56 +0800 - rev 441722
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415397 - use Span<> to replace low level pointer arithmetic in ReadFromCache(). r=bechen,gerald MozReview-Commit-ID: HH6KXtMfSIJ
8a306a5bc1da4a6c1ff7d3cd695b95bdfdccc0f4: Bug 1413427 - Part5 - Fix cross origin test failure. r=bz,jgraham
James Cheng <jacheng@mozilla.com> - Fri, 03 Nov 2017 17:57:52 +0800 - rev 441721
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1413427 - Part5 - Fix cross origin test failure. r=bz,jgraham Bug 1322517 will make the EME APIs only run on secure context. This bug will try to make WPT test run with https. According to the spec https://www.w3.org/TR/secure-contexts/#is-url-trustworthy I got "TypeError: navigator.requestMediaKeySystemAccess is not a function" by the original code. So I need this patch to make the test work as expected. MozReview-Commit-ID: Gp23IcscXHE
6039726995022771be1e7a853fcc238f9fbac829: Bug 1413427 - Part4 - Rename the content of the .ini file and update the MANIFEST.JSON files by command. r=jgraham
James Cheng <jacheng@mozilla.com> - Fri, 03 Nov 2017 21:17:29 +0800 - rev 441720
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1413427 - Part4 - Rename the content of the .ini file and update the MANIFEST.JSON files by command. r=jgraham MozReview-Commit-ID: It2X2FG8iS1
294ccf3ffc059ff6a206291715bcaa363a0cee0d: Bug 1413427 - Part3 - Change the file extension from .html.ini to .https.html.ini. r=jgraham
James Cheng <jacheng@mozilla.com> - Fri, 03 Nov 2017 21:20:25 +0800 - rev 441719
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1413427 - Part3 - Change the file extension from .html.ini to .https.html.ini. r=jgraham MozReview-Commit-ID: LnJkmoyTYX5
047b40ad4a46d25e5b985e47dde46cbeb52260aa: Bug 1413427 - Part2 - Change the file extension from .html to .https.html. r=jgraham
James Cheng <jacheng@mozilla.com> - Fri, 03 Nov 2017 21:23:35 +0800 - rev 441718
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1413427 - Part2 - Change the file extension from .html to .https.html. r=jgraham MozReview-Commit-ID: IfLOOM85WyL
c7c230ee775e023a243f2a80601b0f7b4d80188e: Bug 1413427 - Part1 - Make Mochitest runs EME with https. r=cpearce,kikuo
James Cheng <jacheng@mozilla.com> - Fri, 03 Nov 2017 12:30:58 +0800 - rev 441717
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1413427 - Part1 - Make Mochitest runs EME with https. r=cpearce,kikuo MozReview-Commit-ID: 8EfJTXdQkZG
bfa36a25eaf674064cf16bdcd8ef4eb049359591: Bug 1415454 - Replace log2 lookup table with FloorLog2. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Wed, 08 Nov 2017 16:20:40 +0900 - rev 441716
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415454 - Replace log2 lookup table with FloorLog2. r=njn FloorLog2 expands to, essentially, a compiler builtin/intrinsic, that, in turn, expands to a single machine instruction on tier 1 and other platforms. On platforms where that's not the case, we can expect the compiler to generate fast code anyways. So overall, this is all better than manually using a log2 lookup table. Also replace a manual power-of-two check with mozilla::IsPowerOfTwo, which does the same test.
b8acdda178182e9dec084f84af572294d6d58ac6: Bug 1415454 - Remove the unused arena_bin_t* argument to arena_t::AllocRun. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 03 Nov 2017 15:54:20 +0900 - rev 441715
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415454 - Remove the unused arena_bin_t* argument to arena_t::AllocRun. r=njn
f64defe0d11edb2a76bb6cca295a7fd864542501: Bug 1415454 - Inline MallocBinEasy and MallocBinHard. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 03 Nov 2017 15:48:40 +0900 - rev 441714
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1415454 - Inline MallocBinEasy and MallocBinHard. r=njn They are both only used once, are trivial wrappers, and even repeat the same assertions.
78acf48d63c3dacd8885ba93cdf5f1b2efccafe5: Bug 1403881 - Add credit card update doorhanger. r=lchang
steveck-chung <schung@mozilla.com> - Tue, 07 Nov 2017 16:50:35 +0800 - rev 441713
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1403881 - Add credit card update doorhanger. r=lchang MozReview-Commit-ID: BUd70UyLC9c
a4392df95024d106fb3472b746c96e31224cdee4: Bug 1414168 - Rename arena_run_t fields. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 03 Nov 2017 15:23:44 +0900 - rev 441712
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1414168 - Rename arena_run_t fields. r=njn
182b840c89560a0d4a53bcd5bc337749e92fdd23: Bug 1414168 - Move bin initialization to a method of the arena_bin_t class. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Wed, 08 Nov 2017 15:53:24 +0900 - rev 441711
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1414168 - Move bin initialization to a method of the arena_bin_t class. r=njn
11bc33786fd378b717fc4ea358c12e6f96320432: Bug 1414168 - Change how run sizes are calculated. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Wed, 08 Nov 2017 14:04:10 +0900 - rev 441710
Push 8133 by ryanvm@gmail.com at Thu, 09 Nov 2017 23:33:01 +0000
Bug 1414168 - Change how run sizes are calculated. r=njn There are multiple flaws to the current code: - The loop calculating the right parameters for a given run size is repeated. - The loop trying different run sizes doesn't actually work to fulfil the overhead constraint: while it stops when the constraint is fulfilled, the values that are kept are those from the previous iteration, which may well be well over the constraint. In practice, the latter resulted in a few surprising results: - most size classes had an overhead slightly over the constraint (1.562%), which, while not terribly bad, doesn't match the set expectations. - some size classes ended up with relatively good overheads only because of the additional constraint that run sizes had to be larger than the run size of smaller size classes. Without this constraint, some size classes would end up with overheads well over 2% just because that happens to be the last overhead value before reaching below the 1.5% constraint. Furthermore, for higher-level fragmentation concerns, smaller run sizes are better than larger run sizes, and in many cases, smaller run sizes can yield the same (or even sometimes, better) overhead as larger run sizes. For example, the current code choses 8KiB for runs of size 112, but using 4KiB runs would actually yield the same number of regions, and the same overhead. We thus change the calculation to: - not force runs to be smaller than those of smaller classes. - avoid the code repetition. - actually enforce its overhead constraint, but make it 1.6%. - for especially small size classes, relax the overhead constraint to 2.4%. This leads to an uneven set of run sizes: size class before after 4 4 KiB 4 KiB 8 4 KiB 4 KiB 16 4 KiB 4 KiB 32 4 KiB 4 KiB 48 4 KiB 4 KiB 64 4 KiB 4 KiB 80 4 KiB 4 KiB 96 4 KiB 4 KiB 112 8 KiB 4 KiB 128 8 KiB 8 KiB 144 8 KiB 4 KiB 160 8 KiB 8 KiB 176 8 KiB 4 KiB 192 12 KiB 4 KiB 208 12 KiB 8 KiB 224 12 KiB 4 KiB 240 12 KiB 4 KiB 256 16 KiB 16 KiB 272 16 KiB 4 KiB 288 16 KiB 4 KiB 304 16 KiB 12 KiB 320 20 KiB 12 KiB 336 20 KiB 4 KiB 352 20 KiB 8 KiB 368 20 KiB 4 KiB 384 24 KiB 8 KiB 400 24 KiB 20 KiB 416 24 KiB 16 KiB 432 24 KiB 12 KiB 448 28 KiB 4 KiB 464 28 KiB 16 KiB 480 28 KiB 8 KiB 496 28 KiB 20 KiB 512 32 KiB 32 KiB 1024 64 KiB 64 KiB 2048 132 KiB 128 KiB * Note: before is before this change only, not before the set of changes from this bug; before that, the run size for 96 could be 8 KiB in some configurations. In most cases, the overhead hasn't changed, with a few exceptions: - Improvements: size class before after 208 1.823% 0.977% 304 1.660% 1.042% 320 1.562% 1.042% 400 0.716% 0.391% 464 1.283% 0.879% 480 1.228% 0.391% 496 1.395% 0.703% - Regressions: 352 0.312% 1.172% 416 0.130% 0.977% 2048 1.515% 1.562% For the regressions, the values are either still well within the constraint or very close to the previous value, that I don't feel like it's worth trying to avoid them, with the risk of making things worse for other size classes.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip