searching for reviewer(heycam)
2bbc29a1d8f1e6f412c2084a361d098cb5f7b673: Bug 1686106 - Allow the character map record created for unicode-range to be shared among FontFaces with the same descriptor. r=heycam
Jonathan Kew <jkew@mozilla.com> - Tue, 12 Jan 2021 22:07:10 +0000 - rev 562843
Push 134033 by jkew@mozilla.com at Tue, 12 Jan 2021 22:10:14 +0000
Bug 1686106 - Allow the character map record created for unicode-range to be shared among FontFaces with the same descriptor. r=heycam This will reduce memory footprint when there are multiple @font-face rules with the same unicode-range descriptor, which is common when, for example, a site loads several styles of a family, all with the same character repertoire. Differential Revision: https://phabricator.services.mozilla.com/D101411
28268df8154dad7a2e686ff95ad5559a46316cbf: Bug 1685675 - Ship :autofill / :-webkit-autofill. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 12 Jan 2021 02:52:56 +0000 - rev 562757
Push 133968 by ealvarez@mozilla.com at Tue, 12 Jan 2021 04:35:31 +0000
Bug 1685675 - Ship :autofill / :-webkit-autofill. r=heycam Spec PR at https://github.com/whatwg/html/pull/6273 Tests at https://github.com/web-platform-tests/wpt/pull/27098 Differential Revision: https://phabricator.services.mozilla.com/D101148
014013fe5e4503c130697d48dc43e239c8cedb27: Bug 1685950 - update cssparser. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 11 Jan 2021 02:11:11 +0000 - rev 562610
Push 133861 by ealvarez@mozilla.com at Mon, 11 Jan 2021 02:13:40 +0000
Bug 1685950 - update cssparser. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D101300
6326153dee82fd68c6332be87bfb3db15078811d: Bug 1685917 - Remove layout.css.unknown-webkit-pseudo-element. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 11 Jan 2021 01:53:12 +0000 - rev 562609
Push 133860 by ealvarez@mozilla.com at Mon, 11 Jan 2021 02:07:55 +0000
Bug 1685917 - Remove layout.css.unknown-webkit-pseudo-element. r=heycam Enabled since bug 1486325 (Firefox 64). Depends on D101290 Differential Revision: https://phabricator.services.mozilla.com/D101291
b0e30bc705203337aebcfa3b58774080bb6c228f: Bug 1685917 - Remove layout.css.file-selector-button.enabled pref. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 11 Jan 2021 01:55:50 +0000 - rev 562608
Push 133860 by ealvarez@mozilla.com at Mon, 11 Jan 2021 02:07:55 +0000
Bug 1685917 - Remove layout.css.file-selector-button.enabled pref. r=heycam On by default since bug 1662478 (Firefox 82) Depends on D101289 Differential Revision: https://phabricator.services.mozilla.com/D101290
36933e4cd60d1c3cb4b4a6793f660448e1fd62cb: Bug 1685917 - Remove the moz-focus-outer code. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 11 Jan 2021 01:57:17 +0000 - rev 562607
Push 133860 by ealvarez@mozilla.com at Mon, 11 Jan 2021 02:07:55 +0000
Bug 1685917 - Remove the moz-focus-outer code. r=heycam This pseudo does nothing and we successfully removed it in bug 1655859 (firefox 81). Differential Revision: https://phabricator.services.mozilla.com/D101289
e202f996ca42da4e343446bbb7e5b8dbbf7f3ac7: Bug 1684497 - Hoist ResetGlyphRuns out of the conditional statement. r=heycam
Jonathan Kew <jkew@mozilla.com> - Tue, 05 Jan 2021 21:53:40 +0000 - rev 562203
Push 133659 by rvandermeulen@mozilla.com at Wed, 06 Jan 2021 22:20:34 +0000
Bug 1684497 - Hoist ResetGlyphRuns out of the conditional statement. r=heycam This matches how it's done at the other callsite. (Also add an assertion that would've caught the omission sooner.) Differential Revision: https://phabricator.services.mozilla.com/D100779
e38d11902d246520aa55af8fa8cabf167995df3c: Bug 1683295 - Make next_in_preorder generate slightly better code. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 23 Dec 2020 02:59:11 +0000 - rev 561691
Push 133319 by ealvarez@mozilla.com at Wed, 23 Dec 2020 05:45:31 +0000
Bug 1683295 - Make next_in_preorder generate slightly better code. r=heycam This avoids the panic code in release builds. Differential Revision: https://phabricator.services.mozilla.com/D100094
0e4443a87251825eee827c87b13aa00fbb890db7: Bug 1682882 - Don't crash when trying to figure out the parent style of a node about to be reframed. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 23 Dec 2020 03:01:19 +0000 - rev 561690
Push 133318 by ealvarez@mozilla.com at Wed, 23 Dec 2020 05:44:49 +0000
Bug 1682882 - Don't crash when trying to figure out the parent style of a node about to be reframed. r=heycam This is basically the situation described here: https://searchfox.org/mozilla-central/rev/8d722de75886d6bffc116772a1db8854e34ee6a7/layout/generic/nsIFrame.cpp#9961-9964 Differential Revision: https://phabricator.services.mozilla.com/D100097
f90f082c3a8b24e96330b81c3e2db336ba25a823: Bug 1682988 - Use UTF8String for querySelector and similar OM methods. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 23 Dec 2020 05:42:00 +0000 - rev 561689
Push 133317 by ealvarez@mozilla.com at Wed, 23 Dec 2020 05:44:23 +0000
Bug 1682988 - Use UTF8String for querySelector and similar OM methods. r=heycam This has no observable behavior change because we were converting to UTF-8 in ParseSelectorList. Differential Revision: https://phabricator.services.mozilla.com/D99961
b2cf325325e1ae08de0e387015ae2b0ac02a18a8: Bug 1682003 - Avoid UTF-8 -> UTF-16 conversion during CSSOM serialization. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 17 Dec 2020 14:04:35 +0000 - rev 561121
Push 133023 by ealvarez@mozilla.com at Thu, 17 Dec 2020 14:08:49 +0000
Bug 1682003 - Avoid UTF-8 -> UTF-16 conversion during CSSOM serialization. r=heycam This lifts a bunch of string conversions higher up the stack, but allows us to make the servo code use utf-8 unconditionally, and seemed faster in my benchmarking (see comment 0). It should also make a bunch of attribute setters faster too (like setting .cssText), now that we use UTF8String for them (we couldn't because we couldn't specify different string types for the getter and setters). Differential Revision: https://phabricator.services.mozilla.com/D99590
fd07538e12316562b6479e77af02c7b2b0f94d43: Bug 1577530 - Cache theming values from nsTextControlFrame correctly. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 16 Dec 2020 21:23:17 +0000 - rev 560999
Push 132980 by ealvarez@mozilla.com at Wed, 16 Dec 2020 21:25:56 +0000
Bug 1577530 - Cache theming values from nsTextControlFrame correctly. r=heycam The issue is that nsTextControlFrame might have a different used padding then the editing root block and thus the editing root's scrolled content, etc, because its padding doesn't match the style. This causes TextOverflow code to miscompute the content area size, which is sadness and causes the overflow markers to go over the padding box. The REFLOW_ROOT condition was introduced in bug 157846 and only partially takes care of the issue. This fix supersedes it, so we can remove it. Differential Revision: https://phabricator.services.mozilla.com/D99838
1fcf791ba732b2d7ef0e899c903e3fa6805c3368: Bug 1470983 - Prelude: Remove use of gtk_widget_get_settings in LookAndFeel. r=heycam
Jed Davis <jld@mozilla.com> - Sun, 13 Dec 2020 01:34:35 +0000 - rev 560899
Push 132911 by cmccormack@mozilla.com at Wed, 16 Dec 2020 05:53:04 +0000
Bug 1470983 - Prelude: Remove use of gtk_widget_get_settings in LookAndFeel. r=heycam The GTK nsLookAndFeel implementation uses `gtk_widget_get_settings` on temporary widgets that aren't attached to anything, which the documentation says not to do. Empirically, this seems to return the same settings object as `gtk_settings_get_default`. Explicitly using `gtk_settings_get_default` is useful for remote look-and-feel, so that the parent process can register for property changes on that object to know when to update the child processes. Differential Revision: https://phabricator.services.mozilla.com/D94535
f6ece5ef6e20e78cc50f4281228e248e8cc63475: Bug 1603455 - Remove layout.css.xul-tree-pseudos.content.enabled. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 14 Dec 2020 13:43:50 +0000 - rev 560587
Push 132704 by ealvarez@mozilla.com at Mon, 14 Dec 2020 13:47:51 +0000
Bug 1603455 - Remove layout.css.xul-tree-pseudos.content.enabled. r=heycam We unshipped these a while ago and left the pref just for testing purposes. But now all the reftests using it were conveniently migrated to chrome:// tests, so we no longer need it. Differential Revision: https://phabricator.services.mozilla.com/D56950
1a65440b05307d39d46e24ef856a1f3eabf5f8b6: Bug 1680084 - Use a more precise rule start for sanitization. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 04 Dec 2020 15:41:24 +0000 - rev 559537
Push 132120 by ealvarez@mozilla.com at Fri, 04 Dec 2020 16:27:16 +0000
Bug 1680084 - Use a more precise rule start for sanitization. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D98710
7fbd27be335d24b67954e6b44701f9ff1cd117cd: Bug 1680558 - Update cssparser to get ParserState in rule parser. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 04 Dec 2020 10:47:55 +0000 - rev 559504
Push 132098 by ealvarez@mozilla.com at Fri, 04 Dec 2020 10:50:26 +0000
Bug 1680558 - Update cssparser to get ParserState in rule parser. r=heycam The changes should be trivial. The third_party changes are up for review in https://github.com/servo/rust-cssparser/pull/277 (and of course I'll land with a bump to 0.28 rather than the override after that gets r+'d). The basic idea is that with this we have the actual start offset of the rule, so we wouldn't include html comments or other invalid stuff we discard during sanitization in bug 1680084. But that's a separate change. Differential Revision: https://phabricator.services.mozilla.com/D98677
54068f0444262ae3d54ece6023a277b64877e7e2: Bug 1680558 - Update cssparser to get ParserState in rule parser. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 04 Dec 2020 03:14:36 +0000 - rev 559385
Push 132078 by ealvarez@mozilla.com at Fri, 04 Dec 2020 03:27:00 +0000
Bug 1680558 - Update cssparser to get ParserState in rule parser. r=heycam The changes should be trivial. The third_party changes are up for review in https://github.com/servo/rust-cssparser/pull/277 (and of course I'll land with a bump to 0.28 rather than the override after that gets r+'d). The basic idea is that with this we have the actual start offset of the rule, so we wouldn't include html comments or other invalid stuff we discard during sanitization in bug 1680084. But that's a separate change. Differential Revision: https://phabricator.services.mozilla.com/D98677
f79d16c80de6ec04c618f8bfeb255e22c4ed8462: Bug 1679208 - Fix pre-existing unit confusion in gtk scrollbar painting. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 30 Nov 2020 21:34:09 +0000 - rev 558839
Push 131776 by ealvarez@mozilla.com at Mon, 30 Nov 2020 21:37:14 +0000
Bug 1679208 - Fix pre-existing unit confusion in gtk scrollbar painting. r=heycam This is very visible with the previous changes in the bug. Differential Revision: https://phabricator.services.mozilla.com/D98240
03d5d86b5073b719542b36b281b5ecad2264898e: Bug 1679208 - Use more strongly typed rects in nsNativeBasicTheme. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 29 Nov 2020 22:19:10 +0000 - rev 558634
Push 131702 by ealvarez@mozilla.com at Mon, 30 Nov 2020 04:52:13 +0000
Bug 1679208 - Use more strongly typed rects in nsNativeBasicTheme. r=heycam As per review comments. Depends on D97937 Differential Revision: https://phabricator.services.mozilla.com/D98099
98e36a94b9e8a22ded5079b9d698ed11ab06a817: Bug 1679208 - Improve non-native theme with fractional scaling. r=heycam
Emilio Cobos Alvarez <emilio@crisal.io> - Mon, 30 Nov 2020 04:48:07 +0000 - rev 558633
Push 131702 by ealvarez@mozilla.com at Mon, 30 Nov 2020 04:52:13 +0000
Bug 1679208 - Improve non-native theme with fractional scaling. r=heycam This greatly improves non-native theme on Windows in presence of fractional DPI (see bug 1657191). This matches what the Windows native theme does, to my knowledge, see ScaleForFrameDPI in nsNativeThemeWin.h. However I think the right fix is to not make these LayoutDeviceInt* things but LayoutDevice*. That's a bigger fix, and fits well as a follow-up to this bug, so let's land this in the meantime. Differential Revision: https://phabricator.services.mozilla.com/D97937
7f66d0b85a4c1e5dbb16d0370b447d2b4f00f9db: Bug 1675015: Add Windows high contrast support to non-native theme. r=heycam
Stephen A Pohl <spohl.mozilla.bugs@gmail.com> - Tue, 17 Nov 2020 20:53:01 +0000 - rev 557638
Push 131178 by spohl@mozilla.com at Tue, 17 Nov 2020 21:52:18 +0000
Bug 1675015: Add Windows high contrast support to non-native theme. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D96803
b9313e6a6e2e730922129158cdc138547229f45b: Bug 1677187 - ignore x values for vertical and y values for horizontal text in textPaths r=heycam
longsonr <longsonr@gmail.com> - Mon, 16 Nov 2020 10:54:56 +0000 - rev 557316
Push 131003 by longsonr@gmail.com at Mon, 16 Nov 2020 10:59:31 +0000
Bug 1677187 - ignore x values for vertical and y values for horizontal text in textPaths r=heycam Differential Revision: https://phabricator.services.mozilla.com/D97086
ed3ef7c26759e4a4187c65527e789ccb0fbf773d: Bug 1676843 - Make innerWidth/innerHeight return the actual CSS size rather than a rounded version of it. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 13 Nov 2020 12:44:01 +0000 - rev 557156
Push 130905 by ealvarez@mozilla.com at Fri, 13 Nov 2020 14:45:44 +0000
Bug 1676843 - Make innerWidth/innerHeight return the actual CSS size rather than a rounded version of it. r=heycam On Nightly, for now. This allows websites to get the real viewport size. The rounded size has caused problems in the past (see bug 1648298 / bug 1648265), and changing it would be ideal. I told the CSSWG that we can experiment with this on Nightly to alleviate (or prove) compat concerns, in which case we'd need to use a different solution, see https://github.com/w3c/csswg-drafts/issues/5260. Differential Revision: https://phabricator.services.mozilla.com/D96826
5cfe11c0659ad92715a8f4eebbda67076b737b70: Bug 1676543 - Remove some invalid assertions. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 13 Nov 2020 05:36:37 +0000 - rev 557121
Push 130897 by ealvarez@mozilla.com at Fri, 13 Nov 2020 10:25:54 +0000
Bug 1676543 - Remove some invalid assertions. r=heycam We might clear this flag in PostRebuildAllStyleData before actually flushing style, so if the timing is wrong you can hit this assertion. This doesn't impact correctness in any way that I can see. Differential Revision: https://phabricator.services.mozilla.com/D96685
33e2addbf47ef312da1ccdecc4f3f9db22fccd74: Bug 1634437 - Dispatch NativeInit as an idle runnable on Linux r=heycam
sefeng <sefeng@mozilla.com> - Thu, 12 Nov 2020 20:04:53 +0000 - rev 557023
Push 130860 by sefeng@mozilla.com at Thu, 12 Nov 2020 23:17:14 +0000
Bug 1634437 - Dispatch NativeInit as an idle runnable on Linux r=heycam EnsureInit takes a long time to run on Linux, in addition to it's timing, it also runs within a critical path of MSG_ConstructBrowser. Scheduling it as an idle runnable gives it a chance to run before MSG_ConstructBrowser. Differential Revision: https://phabricator.services.mozilla.com/D83230
017aa3b39d73685dbd8a4010b5e556a2b15999f0: Bug 1675853 - FontFace.load() doesn't need to flush the user fontset. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 09 Nov 2020 07:51:42 +0000 - rev 556414
Push 130486 by ealvarez@mozilla.com at Mon, 09 Nov 2020 12:05:28 +0000
Bug 1675853 - FontFace.load() doesn't need to flush the user fontset. r=heycam This one is even more subtle. This could cause a call from SetStatus() to OnFontFaceStatusChange() to not be done as part of the load() function (if the face is about to get added to the set or such). However, when it's actually added via UpdateRules, we'd properly invalidate the relevant state as well, so I think this is also fine, and our existing tests also agree. Depends on D96342 Differential Revision: https://phabricator.services.mozilla.com/D96343
a60436958371c170c23ecf9be8457251ac72b99d: Bug 1675950 - Check for explicit restyles posted to pseudo-elements to check whether we need a style flush in getComputedStyle. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 09 Nov 2020 11:59:31 +0000 - rev 556413
Push 130485 by ealvarez@mozilla.com at Mon, 09 Nov 2020 12:02:04 +0000
Bug 1675950 - Check for explicit restyles posted to pseudo-elements to check whether we need a style flush in getComputedStyle. r=heycam Font loading invalidation may invalidate only pseudo-elements, like it happens on this test-case. Once we get to testing first-line / first-letter in the test-case (css/css-values/ch-pseudo-recalc-on-font-load.html), we end up flushing because those end up posting the restyle to the parent element correctly. Differential Revision: https://phabricator.services.mozilla.com/D96373
00cf70afee3f9d91ff8e0ebe7ba0b1cdad9bd785: Bug 1675950 - Check for explicit restyles posted to pseudo-elements to check whether we need a style flush in getComputedStyle. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 09 Nov 2020 06:25:41 +0000 - rev 556382
Push 130460 by ealvarez@mozilla.com at Mon, 09 Nov 2020 06:31:36 +0000
Bug 1675950 - Check for explicit restyles posted to pseudo-elements to check whether we need a style flush in getComputedStyle. r=heycam Font loading invalidation may invalidate only pseudo-elements, like it happens on this test-case. Once we get to testing first-line / first-letter in the test-case (css/css-values/ch-pseudo-recalc-on-font-load.html), we end up flushing because those end up posting the restyle to the parent element correctly. Differential Revision: https://phabricator.services.mozilla.com/D96373
9e2f1e514e8495b8af1560ff7013771e8420a086: Bug 1675853 - FontFace.load() doesn't need to flush the user fontset. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 09 Nov 2020 04:12:52 +0000 - rev 556380
Push 130458 by ealvarez@mozilla.com at Mon, 09 Nov 2020 05:50:22 +0000
Bug 1675853 - FontFace.load() doesn't need to flush the user fontset. r=heycam This one is even more subtle. This could cause a call from SetStatus() to OnFontFaceStatusChange() to not be done as part of the load() function (if the face is about to get added to the set or such). However, when it's actually added via UpdateRules, we'd properly invalidate the relevant state as well, so I think this is also fine, and our existing tests also agree. Depends on D96342 Differential Revision: https://phabricator.services.mozilla.com/D96343
30d5f7d7ab51ab63c4f39d37107f8c774b9b5cdb: Bug 1676083 - Use font-variant-numeric in the video control duration. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 09 Nov 2020 05:42:49 +0000 - rev 556379
Push 130457 by ealvarez@mozilla.com at Mon, 09 Nov 2020 05:45:53 +0000
Bug 1676083 - Use font-variant-numeric in the video control duration. r=heycam If you keep the controls showing on a video, depending on your fonts you can see the progress bar growing and shrinking depending on the current duration value. This is more noticeable at high video speed. We can use font-variant-numeric: tabular-nums; to make Gecko choose the right glyphs if available so that the size doesn't change over time. Differential Revision: https://phabricator.services.mozilla.com/D96370
e7714e0e74a31463474927a28aec269c149c6bbb: Bug 1675853 - FontFace getters don't need to flush the fontset. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 09 Nov 2020 03:18:46 +0000 - rev 556374
Push 130453 by ealvarez@mozilla.com at Mon, 09 Nov 2020 04:11:51 +0000
Bug 1675853 - FontFace getters don't need to flush the fontset. r=heycam There's no way these need to do a flush as they just return data from the descriptors etc. Similarly .loaded also doesn't rely on the font face set. Differential Revision: https://phabricator.services.mozilla.com/D96341
147c3a29af7d68f69c8004ca417b29a1ec2d2538: [css-grid-3] Bug 1665152 - Add more editors to the CSS Masonry draft spec. r=heycam
Mats Palmgren <mats@mozilla.com> - Tue, 20 Oct 2020 22:15:01 +0000 - rev 553731
Push 129039 by mpalmgren@mozilla.com at Tue, 20 Oct 2020 22:20:06 +0000
[css-grid-3] Bug 1665152 - Add more editors to the CSS Masonry draft spec. r=heycam Per CSSWG meeting at TPAC: https://www.w3.org/2020/10/20-css-irc NPOTB DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D94204
40bc2dbc5201def6aebc34113cd9c422c82f85ff: Bug 1668875 - Invalidate images for media query changes more granularly. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 20 Oct 2020 21:24:30 +0000 - rev 553726
Push 129035 by ealvarez@mozilla.com at Tue, 20 Oct 2020 21:32:45 +0000
Bug 1668875 - Invalidate images for media query changes more granularly. r=heycam The SVG code invalidating too much causes hangs with pages that have lots of SVGs. Differential Revision: https://phabricator.services.mozilla.com/D94186
1373b2edc5799db73d1b6beeb066e4eddcdf0a3a: Bug 1671863 - Skip reftest on mingw platform. r=heycam
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 20 Oct 2020 02:46:28 +0000 - rev 553517
Push 128958 by jyavenard@mozilla.com at Tue, 20 Oct 2020 02:49:20 +0000
Bug 1671863 - Skip reftest on mingw platform. r=heycam The RDD doesn't start on MinGW and so the AV1 decoder isn't available. Differential Revision: https://phabricator.services.mozilla.com/D93909
3cb57a090de2c5c436bc9bab230e365c35b46ee7: Bug 1490969 - Remove intermittent annotations on mask-image tests, r=heycam,mats,TYLin,emilio
James Graham <james@hoppipolla.co.uk> - Fri, 16 Oct 2020 13:30:23 +0000 - rev 553308
Push 128820 by james@hoppipolla.co.uk at Fri, 16 Oct 2020 16:48:36 +0000
Bug 1490969 - Remove intermittent annotations on mask-image tests, r=heycam,mats,TYLin,emilio I couldn't reproduce this bug with a try push containing 20 retriggers, so I think it's safe to remove the annotations. If this comes up again as an intermittent, we can revert this patch and investigate further. Differential Revision: https://phabricator.services.mozilla.com/D93126
2127a58b2f0db518a2b5c2f3a40b842f53fdc499: Bug 1484928 - Add an use counter for when people mix content: url() and <img src>. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 10 Oct 2020 13:41:08 +0000 - rev 553017
Push 128644 by ealvarez@mozilla.com at Wed, 14 Oct 2020 15:24:06 +0000
Bug 1484928 - Add an use counter for when people mix content: url() and <img src>. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D93071
4b2f994f3fe2cad3895cc4d5044a3466a2d5eda4: Bug 1668253 - Convert hyphenation from ipc::SharedMemoryBasic to base::SharedMemory, and freeze blocks before sharing to content processes. r=heycam
Jonathan Kew <jkew@mozilla.com> - Mon, 12 Oct 2020 10:24:06 +0000 - rev 552681
Push 128431 by jkew@mozilla.com at Mon, 12 Oct 2020 10:29:42 +0000
Bug 1668253 - Convert hyphenation from ipc::SharedMemoryBasic to base::SharedMemory, and freeze blocks before sharing to content processes. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D91945
bc5b6dcb7057cf610efe29e14b65e9c3aa34d1c3: Bug 1669162 - Update mapped_hyph to 0.4.2 so that .dic parse errors are non-fatal. r=heycam
Jonathan Kew <jkew@mozilla.com> - Mon, 12 Oct 2020 10:23:27 +0000 - rev 552680
Push 128430 by jkew@mozilla.com at Mon, 12 Oct 2020 10:28:48 +0000
Bug 1669162 - Update mapped_hyph to 0.4.2 so that .dic parse errors are non-fatal. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D92435
02be727b0bb569b22aa9217e76cfdd70dbe8743c: Bug 1669459 - Introduce InspectorUtils.supports(). r=rcaliman,heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 09 Oct 2020 03:20:34 +0000 - rev 552382
Push 128321 by ealvarez@mozilla.com at Fri, 09 Oct 2020 11:21:29 +0000
Bug 1669459 - Introduce InspectorUtils.supports(). r=rcaliman,heycam Which allows to run CSS.supports in chrome/user-agent context. We should probably add a couple more tweaks for stuff like quirks or what not (the `ParsingMode` enum), but that seem lower priority. I only added the one-value version of CSS.supports because it should be enough and supporting the two value version required a bit of awkward code due to when we parse the CSS property name right now. Differential Revision: https://phabricator.services.mozilla.com/D92585
520f5faf3a332aea36f3456a34d85655bdcceea9: Bug 1669459 - Clean-up CSS.supports. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Oct 2020 23:56:14 +0000 - rev 552350
Push 128297 by ealvarez@mozilla.com at Fri, 09 Oct 2020 03:22:34 +0000
Bug 1669459 - Clean-up CSS.supports. r=heycam Use UTF8String rather than converting on the callee (which can avoid copies), and also it doesn't really throw, so don't claim it does. Differential Revision: https://phabricator.services.mozilla.com/D92583
e6b1234900b328782dd4f93b34222bf49b470ac2: Bug 1669600 - Properly distinguish between empty children and no children in rule iterator. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Oct 2020 05:39:33 +0000 - rev 552069
Push 128204 by ealvarez@mozilla.com at Thu, 08 Oct 2020 09:45:52 +0000
Bug 1669600 - Properly distinguish between empty children and no children in rule iterator. r=heycam So that skip_children(), which just pops the stack and is used by the dynamic media query evaluation code, works as it should. Differential Revision: https://phabricator.services.mozilla.com/D92717
3466a6111998574ca89803c6641e04ea5f6455f1: Bug 1665773 - Ensure CompressedGlyph::SetComplex() always clears the glyphCount to zero; let gfxShapedText::SetDetailedGlyphs() own the setting of this field. r=heycam
Jonathan Kew <jkew@mozilla.com> - Wed, 30 Sep 2020 21:16:44 +0000 - rev 550989
Push 127602 by jkew@mozilla.com at Wed, 30 Sep 2020 23:47:46 +0000
Bug 1665773 - Ensure CompressedGlyph::SetComplex() always clears the glyphCount to zero; let gfxShapedText::SetDetailedGlyphs() own the setting of this field. r=heycam We also rearrange bits in the CompressedGlyph so that the advance field doesn't need to be shifted, and remove unnecessary glyphCount parameters from various setter methods. Differential Revision: https://phabricator.services.mozilla.com/D90678
ad5cf7320076c263062646d42f6d09cbaaa15d97: Bug 1667192 - patch 3 - For Android, package the uncompiled versions of hyphenation files. r=heycam
Jonathan Kew <jkew@mozilla.com> - Wed, 30 Sep 2020 12:52:40 +0000 - rev 550917
Push 127550 by jkew@mozilla.com at Wed, 30 Sep 2020 13:50:56 +0000
Bug 1667192 - patch 3 - For Android, package the uncompiled versions of hyphenation files. r=heycam This reduces the APK size by about 1.4MB. Differential Revision: https://phabricator.services.mozilla.com/D91411
c29703edd736fe68572460bc600baa0ac032fd99: Bug 1667192 - patch 2 - Update nsHyphenator to support loading either precompiled or uncompiled data. r=heycam
Jonathan Kew <jkew@mozilla.com> - Wed, 30 Sep 2020 12:52:22 +0000 - rev 550916
Push 127550 by jkew@mozilla.com at Wed, 30 Sep 2020 13:50:56 +0000
Bug 1667192 - patch 2 - Update nsHyphenator to support loading either precompiled or uncompiled data. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D91410
a1ab26abf3eb645e202d69dffea86398a0ea4563: Bug 1667192 - patch 1.1 - Run `mach vendor rust` to update in-tree mapped_hyph source. r=heycam
Jonathan Kew <jkew@mozilla.com> - Wed, 30 Sep 2020 12:51:54 +0000 - rev 550915
Push 127550 by jkew@mozilla.com at Wed, 30 Sep 2020 13:50:56 +0000
Bug 1667192 - patch 1.1 - Run `mach vendor rust` to update in-tree mapped_hyph source. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D91409
66b86f8b6682a8eac45ac0ceb2447958acf96201: Bug 1667192 - patch 1 - Update to mapped_hyph v0.4.0, to get FFI access to hyphenation table compilation. r=heycam
Jonathan Kew <jkew@mozilla.com> - Wed, 30 Sep 2020 12:51:17 +0000 - rev 550914
Push 127550 by jkew@mozilla.com at Wed, 30 Sep 2020 13:50:56 +0000
Bug 1667192 - patch 1 - Update to mapped_hyph v0.4.0, to get FFI access to hyphenation table compilation. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D91408
d9ce91c9c5c271553bcf49cbcc62d70f14a5e18c: Bug 1658342 - Enable the remoteObjectEmbed pref by default, r=heycam,valentin
Nika Layzell <nika@thelayzells.com> - Mon, 28 Sep 2020 22:06:44 +0000 - rev 550720
Push 127451 by dluca@mozilla.com at Tue, 29 Sep 2020 11:05:02 +0000
Bug 1658342 - Enable the remoteObjectEmbed pref by default, r=heycam,valentin This also disables the cross-origin resource timing test when Fission is enabled, as resource timing is is broken for cross-origin frames (bug 1658097). Differential Revision: https://phabricator.services.mozilla.com/D91326
49c71a08f0e30e93979b3b30c85375be23a3f3a9: Bug 1658342 - Enable the remoteObjectEmbed pref by default, r=heycam,valentin
Nika Layzell <nika@thelayzells.com> - Mon, 28 Sep 2020 22:06:44 +0000 - rev 550684
Push 127435 by nlayzell@mozilla.com at Tue, 29 Sep 2020 06:56:49 +0000
Bug 1658342 - Enable the remoteObjectEmbed pref by default, r=heycam,valentin This also disables the cross-origin resource timing test when Fission is enabled, as resource timing is is broken for cross-origin frames (bug 1658097). Differential Revision: https://phabricator.services.mozilla.com/D91326
9bcb053caba37dd041391dbeea2020f0638956df: Bug 1659227 - Handle SVG{Inner,Outer}Frame as the clip frame for partial pre-render. r=heycam,botond
Hiroyuki Ikezoe <hikezoe.birchill@mozilla.com> - Wed, 23 Sep 2020 06:37:31 +0000 - rev 549764
Push 126988 by hikezoe.birchill@mozilla.com at Wed, 23 Sep 2020 06:40:20 +0000
Bug 1659227 - Handle SVG{Inner,Outer}Frame as the clip frame for partial pre-render. r=heycam,botond In SVG frame tree SVG{Inner,Outer}Frame is normally not scrollable but clips descendants, thus nsLayoutUtils::GetNearestScrollableFrame is not able to find the clip/scroll frame. So in this change, we introduce GetNearestOverflowClipFrame which is a similar function to GetNearestScrollableFrame but it also checks whether the frame is SVGInnerFrame or SVGOuterFrame which is clipping the given nsIFrame during walking up the frame tree. In this change we also change the logic to get the clip/scroll frame and the scroll id as follows; 1) Get the nearest overflow clip frame in the same document 2) Use the root frame if there is no clip frame 3) Then try to find the scroll id if the clip frame is asynchronously scrollable so that it becomes clearer before. It's still messy though. Differential Revision: https://phabricator.services.mozilla.com/D90647
c8657e9dd191875a827b7df23f5272a786f60ce7: Bug 1665258 - Shrink `CSSEquivTable`. r=heycam
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 23 Sep 2020 01:08:21 +0000 - rev 549746
Push 126977 by nnethercote@mozilla.com at Wed, 23 Sep 2020 04:16:08 +0000
Bug 1665258 - Shrink `CSSEquivTable`. r=heycam Better field packing reduces it from 48 bytes to 40 bytes on 64-bit. This reduces the memory used by each process by 336 bytes on 64-bit. Differential Revision: https://phabricator.services.mozilla.com/D90344