searching for reviewer(jfkthame)
fbcaf33d96a4bc9a8880edb22f6d5e59908db16a: Bug 1716045 - Unify font-family storage. r=mats,jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 18 Jun 2021 19:35:50 +0000 - rev 583814
Push 38552 by abutkovits@mozilla.com at Sat, 19 Jun 2021 08:35:56 +0000
Bug 1716045 - Unify font-family storage. r=mats,jfkthame This changes font-family storage to reuse the rust types, removing a bunch of code while at it. This allows us to, for example, use a single static font family for -moz-bullet and clone it, rather than creating a lot of expensive copies. Differential Revision: https://phabricator.services.mozilla.com/D118011
85a42db205761d2b3843c17ae54892e220bb7e7c: Bug 1716045 - Optimize ArcSlice equality operator for pointer equality. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 18 Jun 2021 19:35:49 +0000 - rev 583813
Push 38552 by abutkovits@mozilla.com at Sat, 19 Jun 2021 08:35:56 +0000
Bug 1716045 - Optimize ArcSlice equality operator for pointer equality. r=jfkthame Just drive-by, but seems reasonable to avoid the per-element comparison if the pointers are the same. Depends on D118015 Differential Revision: https://phabricator.services.mozilla.com/D118016
a85091ff664f72c2b0281123594b0fec9d80a75f: Bug 1716045 - Implement ArcSlice copy/move-assignment operators. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 18 Jun 2021 19:35:49 +0000 - rev 583812
Push 38552 by abutkovits@mozilla.com at Sat, 19 Jun 2021 08:35:56 +0000
Bug 1716045 - Implement ArcSlice copy/move-assignment operators. r=jfkthame We're going to use them. Differential Revision: https://phabricator.services.mozilla.com/D118015
fc9009240a43b9c323a1dc59bc4b8e1f4c61fdb7: Bug 1542807 part 4 - Exclude fonts loaded from User and UserAgent origin sheets from 'document.fonts'. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Mon, 14 Jun 2021 01:22:06 +0000 - rev 582929
Push 38537 by mlaza@mozilla.com at Mon, 14 Jun 2021 09:53:07 +0000
Bug 1542807 part 4 - Exclude fonts loaded from User and UserAgent origin sheets from 'document.fonts'. r=jfkthame Per https://github.com/w3c/csswg-drafts/issues/6126 these were never intended to be included in the first place so this is just fixing a bug. Note that I'm leaving them in the mRuleFaces array so that the font loading machinery works the same as before. I'm just excluding them when queried by document.fonts. Differential Revision: https://phabricator.services.mozilla.com/D111694
e3aee052c4954b53bd9548381fbc61b611a74369: Bug 1542807 part 4 - Exclude fonts loaded from User and UserAgent origin sheets from 'document.fonts'. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Fri, 11 Jun 2021 18:10:39 +0000 - rev 582852
Push 38533 by nerli@mozilla.com at Sat, 12 Jun 2021 09:41:20 +0000
Bug 1542807 part 4 - Exclude fonts loaded from User and UserAgent origin sheets from 'document.fonts'. r=jfkthame Per https://github.com/w3c/csswg-drafts/issues/6126 these were never intended to be included in the first place so this is just fixing a bug. Note that I'm leaving them in the mRuleFaces array so that the font loading machinery works the same as before. I'm just excluding them when queried by document.fonts. Differential Revision: https://phabricator.services.mozilla.com/D111694
baa4d8d38822ebe33f4061b53c8e569e8fdadb09: Bug 1714293 - Add Scots (sco) to language names and accept languages locales, r=jfkthame
Francesco Lodolo (:flod) <flod@lodolo.net> - Thu, 03 Jun 2021 08:25:52 +0000 - rev 581811
Push 38513 by abutkovits@mozilla.com at Fri, 04 Jun 2021 10:21:11 +0000
Bug 1714293 - Add Scots (sco) to language names and accept languages locales, r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D116685
7ceed945f49e5d72f9de2fddd6dbba4fd8d6ca79: Bug 1707096 - fix the build for `clang-trunk`. r=jfkthame
Andi-Bogdan Postelnicu <andi@mozilla.com> - Wed, 02 Jun 2021 14:38:01 +0000 - rev 581709
Push 38511 by cbrindusan@mozilla.com at Wed, 02 Jun 2021 21:44:47 +0000
Bug 1707096 - fix the build for `clang-trunk`. r=jfkthame With D100581 LLVM added support for -Wunused-but-set-parameter and -Wunused-but-set-variable. 1. we shouldn't treat these warning as errors 2. for harfbuzz we apply a patch that has also been upstreamed, #2995, since pragma directives are somehow borken in clang vs gcc. Differential Revision: https://phabricator.services.mozilla.com/D116604
f7944c3f9e5beeacd781e19a05138aff63c993fd: Bug 1711437 - Don't EnsureUniqueInner from the cssRules getter. r=layout-reviewers,jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 25 May 2021 13:23:35 +0000 - rev 580612
Push 38491 by malexandru@mozilla.com at Wed, 26 May 2021 09:48:46 +0000
Bug 1711437 - Don't EnsureUniqueInner from the cssRules getter. r=layout-reviewers,jfkthame Instead, fix up the various content data structures when the stylesheet is mutated. This makes reading a stylesheet not disable style sharing. Differential Revision: https://phabricator.services.mozilla.com/D115203
2c95330ad7c743340b38707b1171363789b9ede3: Bug 1711437 - Don't EnsureUniqueInner from the cssRules getter. r=layout-reviewers,jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 24 May 2021 17:18:44 +0000 - rev 580544
Push 38488 by nbeleuzu@mozilla.com at Mon, 24 May 2021 21:58:32 +0000
Bug 1711437 - Don't EnsureUniqueInner from the cssRules getter. r=layout-reviewers,jfkthame Instead, fix up the various content data structures when the stylesheet is mutated. This makes reading a stylesheet not disable style sharing. Differential Revision: https://phabricator.services.mozilla.com/D115203
49166572ab96ac8abaa294eee3edfc327a369c1b: Bug 1712130 - Make print.print_via_parent a static pref. r=layout-reviewers,jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 20 May 2021 17:05:33 +0000 - rev 580309
Push 38479 by malexandru@mozilla.com at Fri, 21 May 2021 09:57:54 +0000
Bug 1712130 - Make print.print_via_parent a static pref. r=layout-reviewers,jfkthame Differential Revision: https://phabricator.services.mozilla.com/D115612
50c8711162bd0aa7931a425f1bd77e85c1f05ce5: Bug 1711472 - Update HarfBuzz to 2.8.1. r=jfkthame
Ryan VanderMeulen <ryanvm@gmail.com> - Mon, 17 May 2021 15:51:42 +0000 - rev 579782
Push 38468 by apavel@mozilla.com at Mon, 17 May 2021 21:26:00 +0000
Bug 1711472 - Update HarfBuzz to 2.8.1. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D115212
7b01dc039dbce65180b8aee89548aaf1a24eaf89: Bug 1690954 - Draw the menulist arrow button using currentColor rather than a fixed color. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 17 May 2021 12:11:15 +0000 - rev 579759
Push 38468 by apavel@mozilla.com at Mon, 17 May 2021 21:26:00 +0000
Bug 1690954 - Draw the menulist arrow button using currentColor rather than a fixed color. r=jfkthame As its background might not be themed. Differential Revision: https://phabricator.services.mozilla.com/D115194
7a17f213bf4ed7424075c338e40039d86137a23a: Bug 1694104: Add a Mutex to nsPrinterWin and lock around most driver calls. r=jfkthame
Bob Owen <bobowencode@gmail.com> - Fri, 07 May 2021 07:04:38 +0000 - rev 578818
Push 38444 by nbeleuzu@mozilla.com at Fri, 07 May 2021 16:18:55 +0000
Bug 1694104: Add a Mutex to nsPrinterWin and lock around most driver calls. r=jfkthame This also removes the fix from bug 1677388, which worked in some circumstances, but is effectively superseded by this one. Removing that fix also reduces potential contention for copying the default DEVMODE. Differential Revision: https://phabricator.services.mozilla.com/D112635
938a7a66d372b3343893fbf38f151aa1ef3d7bcf: Bug 1709647 - Use NS_SAME_AS_FOREGROUND_COLOR instead of NS_DONT_CHANGE_COLOR. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 06 May 2021 09:57:19 +0000 - rev 578697
Push 38440 by malexandru@mozilla.com at Thu, 06 May 2021 15:49:21 +0000
Bug 1709647 - Use NS_SAME_AS_FOREGROUND_COLOR instead of NS_DONT_CHANGE_COLOR. r=jfkthame I don't think they are meaningfully different. Depends on D114390 Differential Revision: https://phabricator.services.mozilla.com/D114391
2de2d2db54accaeb248b214d2db8022437543aef: Bug 1709647 - Remove unused NS_CHANGE_COLOR_IF_SAME_AS_BG. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 06 May 2021 09:57:19 +0000 - rev 578696
Push 38440 by malexandru@mozilla.com at Thu, 06 May 2021 15:49:21 +0000
Bug 1709647 - Remove unused NS_CHANGE_COLOR_IF_SAME_AS_BG. r=jfkthame It was introduced in bug 1024930, but it is unused since. Differential Revision: https://phabricator.services.mozilla.com/D114389
7e8d6b124d6f2f2a1be5f812b36a124df0d4c49e: Bug 1708285 - Pass default font settings from parent to child processes. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Sat, 01 May 2021 20:00:31 +0000 - rev 578270
Push 38425 by nbeleuzu@mozilla.com at Sun, 02 May 2021 09:29:52 +0000
Bug 1708285 - Pass default font settings from parent to child processes. r=jfkthame Instead of relying on GTK to be initialized in the child process. Differential Revision: https://phabricator.services.mozilla.com/D113784
0aa26695dbb6dfa7ee0cc70852d71cc7b8718d98: Bug 1692676 - Disable gfx.font_rendering.fallback.async for css-counter-styles/myanmar/ tests. r=layout-reviewers,jfkthame
Ting-Yu Lin <tlin@mozilla.com> - Wed, 21 Apr 2021 17:36:24 +0000 - rev 576977
Push 38397 by btara@mozilla.com at Thu, 22 Apr 2021 03:04:50 +0000
Bug 1692676 - Disable gfx.font_rendering.fallback.async for css-counter-styles/myanmar/ tests. r=layout-reviewers,jfkthame Also, adjust test expectations because they are now running reliably. Differential Revision: https://phabricator.services.mozilla.com/D112874
8990a3df0feb2de6426b74fa90e82aa337924a20: Bug 1705826 - Disable skip-ink-multiline-position.html on Windows 7. r=jfkthame
Botond Ballo <botond@mozilla.com> - Wed, 21 Apr 2021 00:07:46 +0000 - rev 576867
Push 38392 by ncsoregi@mozilla.com at Wed, 21 Apr 2021 03:11:09 +0000
Bug 1705826 - Disable skip-ink-multiline-position.html on Windows 7. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D112832
6f7a64d3796bf8acb2421721d8a6b8b94db2a067: Bug 1704466 - Make sure to preserve face order from fontconfig when appropriate. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 20 Apr 2021 10:42:44 +0000 - rev 576746
Push 38391 by ncsoregi@mozilla.com at Tue, 20 Apr 2021 21:39:49 +0000
Bug 1704466 - Make sure to preserve face order from fontconfig when appropriate. r=jfkthame This seems to help: https://treeherder.mozilla.org/perfherder/compare?originalProject=try&originalRevision=158613f137b8fe9409f19a3b6abbe5ce773a7b4f&newProject=try&newRevision=c87de791fbfc5d269d0f2071dc8a6dd28707bebc&framework=13 My patch reversed the ordering of faces with different weight / style / etc for a single font (which shouldn't matter for correctness, but seems to have some perf implications). Based on logging, fontconfig on automation returns "regular" faces first and "bold / semibold / italic / etc" later. My guess is that putting the regular faces later (which are more commonly used) affected performance of these test-cases. This patch should address it and restore the original order, by tweaking the prioritization code to stop doing "insert to front" / "insert to back", but instead moving aliases to the end. Differential Revision: https://phabricator.services.mozilla.com/D112674
a4acaff2d290792ad66d2f9ba00dc90ce6336b81: Bug 1700565 - Disallow spaces to contribute to minimum advance width when line breaking is everywhere. r=jfkthame
Ting-Yu Lin <tlin@mozilla.com> - Tue, 20 Apr 2021 02:32:45 +0000 - rev 576702
Push 38390 by imoraru@mozilla.com at Tue, 20 Apr 2021 09:51:22 +0000
Bug 1700565 - Disallow spaces to contribute to minimum advance width when line breaking is everywhere. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D112255
48a96ab7e13d8fbc8c28024bf0c86156090edbea: Bug 1476524 - Don't require so much contrast for selection background-against-background checks. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 13 Apr 2021 16:29:14 +0000 - rev 575693
Push 38369 by cbrindusan@mozilla.com at Tue, 13 Apr 2021 21:43:14 +0000
Bug 1476524 - Don't require so much contrast for selection background-against-background checks. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D111794
64b1938f0ed6fc36f8e82160d7bb968c5dec7d72: Bug 1704167 - When a pattern ends in whitespace, try to collapse adjacent white-space. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 12 Apr 2021 16:11:21 +0000 - rev 575475
Push 38366 by apavel@mozilla.com at Mon, 12 Apr 2021 21:34:34 +0000
Bug 1704167 - When a pattern ends in whitespace, try to collapse adjacent white-space. r=jfkthame This I'm not 100% sure. Should be harmless, but it's a bit subtle. Maybe we should special-case it with "we're at the beginning of the pattern"? Reasoning below: The previous patch restores the performance of the original test-case. However, if you go to the reduced test-case and try to type " re" in the findbar, we still take a long time. The reason for that is not that the previous patch is not effective, but that the findbar sends find requests as soon as you type, and thus we end up with a request to find " ", which matches a gazillion spaces in the page and causes us to use tons of memory and time. Finding " re" is actually super-fast :-) This fixes it, but it is a bit subtle, so thoughts? Perhaps the findbar should wait a bit to perform the search before sending a query for " " instead or something? But I'd rather make it fast. Differential Revision: https://phabricator.services.mozilla.com/D111634
ac196dc4302596c80be0b409fb04ba3045144352: Bug 1704167 - Don't rewind if we're still at the beginning of the pattern. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 12 Apr 2021 16:04:02 +0000 - rev 575472
Push 38366 by apavel@mozilla.com at Mon, 12 Apr 2021 21:34:34 +0000
Bug 1704167 - Don't rewind if we're still at the beginning of the pattern. r=jfkthame This can happen with whitespace, and can cause us to scan whitespace exponentially. Should be straight-forward and have no behavior change. This restores the performance characteristics of the findbar before the regressing bug. Differential Revision: https://phabricator.services.mozilla.com/D111633
65fcd6b3fd44233efbef557bf22c2606cf8f0da9: Bug 1703693 - Prefer faces for fonts that can only be named in one way. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 09 Apr 2021 14:50:44 +0000 - rev 575239
Push 38359 by nbeleuzu@mozilla.com at Sat, 10 Apr 2021 03:36:58 +0000
Bug 1703693 - Prefer faces for fonts that can only be named in one way. r=jfkthame This produces the desired rendering here by effectively "deprioritizing" faces from fonts that have multiple names. Your call whether this is a good idea or not, seems reasonable off-hand to me. Differential Revision: https://phabricator.services.mozilla.com/D111283
a895e219ce93c27c73aa420a9e395feb3c345282: Bug 1695937 - Switch FluentBundle to use intl::NumberFormat; r=zbraniecki,hsivonen,jfkthame
Dan Minor <dminor@mozilla.com> - Thu, 08 Apr 2021 14:56:33 +0000 - rev 575067
Push 38358 by archaeopteryx@coole-files.de at Fri, 09 Apr 2021 09:20:20 +0000
Bug 1695937 - Switch FluentBundle to use intl::NumberFormat; r=zbraniecki,hsivonen,jfkthame Differential Revision: https://phabricator.services.mozilla.com/D107224
536ffb297e79137443a65c03e3a74d438c616f80: Bug 1698043 - Upstream the test for this bug to WPT. r=layout-reviewers,jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Apr 2021 10:40:51 +0000 - rev 574947
Push 38357 by malexandru@mozilla.com at Thu, 08 Apr 2021 21:54:39 +0000
Bug 1698043 - Upstream the test for this bug to WPT. r=layout-reviewers,jfkthame Seems like a reasonable thing to expect that the placeholder renders at the same place as the text it's a placeholder for. Differential Revision: https://phabricator.services.mozilla.com/D111239
d17059c999a6f275259f566770f9b37b27d5d2ec: Bug 1695937 - Switch FluentBundle to use intl::NumberFormat; r=zbraniecki,hsivonen,jfkthame
Dan Minor <dminor@mozilla.com> - Wed, 07 Apr 2021 20:03:22 +0000 - rev 574876
Push 38356 by archaeopteryx@coole-files.de at Thu, 08 Apr 2021 09:51:11 +0000
Bug 1695937 - Switch FluentBundle to use intl::NumberFormat; r=zbraniecki,hsivonen,jfkthame Differential Revision: https://phabricator.services.mozilla.com/D107224
794801ae0407b0936961d637ccbd215bfde75b0f: Bug 1703222 - Make sure to set gfxFontStyle::systemFont in LookAndFeelFontToStyle. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 06 Apr 2021 13:42:57 +0000 - rev 574516
Push 38351 by btara@mozilla.com at Wed, 07 Apr 2021 03:19:44 +0000
Bug 1703222 - Make sure to set gfxFontStyle::systemFont in LookAndFeelFontToStyle. r=jfkthame All NativeGetFont implementations do, so child processes see something different from parent processes which is pretty unfortunate, and isn't the goal of RemoteLookAndFeel. I found this by code inspection. Seems this is mostly used for debugging etc? Differential Revision: https://phabricator.services.mozilla.com/D110914
4a5976e79e137a26c433adc3bcde8602994ff10d: Bug 1701942 - Use bitwise equality for font variation value comparisons. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 02 Apr 2021 12:37:23 +0000 - rev 574259
Push 38343 by abutkovits@mozilla.com at Fri, 02 Apr 2021 21:41:49 +0000
Bug 1701942 - Use bitwise equality for font variation value comparisons. r=jfkthame Just like we do for font sizes / size-adjust in gfxFontStyle. Differential Revision: https://phabricator.services.mozilla.com/D110519
79a77f6c10d476fd6c198f4e8bf1c3f7c6b8826f: Bug 1701942 - Normalize font size when font inflation is involved. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 02 Apr 2021 12:37:23 +0000 - rev 574258
Push 38343 by abutkovits@mozilla.com at Fri, 02 Apr 2021 21:41:49 +0000
Bug 1701942 - Normalize font size when font inflation is involved. r=jfkthame Bug 1646224 made font sizes floats, which caused conversion to nscoord not happen. Style fonts should never have NaN, but in this case a massive SVG transform matrix ends up causing a NaN font inflation factor. This in turn caused the scaled size to be NaN, and to get stuck on the font variations array here: https://searchfox.org/mozilla-central/rev/4fa18c26fa907f38d56b599571b9846af1506f3c/gfx/src/nsFont.cpp#283 That NaN in turn caused the font variations to have a NaN, and font styles comparing unequal, which prevents the font from getting removed from the cache, causing the issue. This should prevent it, but I have a more in-depth patch incoming. Differential Revision: https://phabricator.services.mozilla.com/D110518
8119f4fe4e541911842ecf511ebd03932a88c177: Bug 1698885 - Update HarfBuzz to 2.8.0. r=jfkthame
Ryan VanderMeulen <ryanvm@gmail.com> - Wed, 31 Mar 2021 15:45:26 +0000 - rev 573895
Push 38337 by ncsoregi@mozilla.com at Wed, 31 Mar 2021 21:54:44 +0000
Bug 1698885 - Update HarfBuzz to 2.8.0. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D109838
ced7356760f04586052b41bf2062a52688060f5d: Bug 1679645 - Cache the selection state in nsTextFrame rather than in nsDisplayText r=emilio,jfkthame
Miko Mynttinen <mikokm@gmail.com> - Thu, 25 Mar 2021 21:28:52 +0000 - rev 573157
Push 38321 by smolnar@mozilla.com at Fri, 26 Mar 2021 04:08:09 +0000
Bug 1679645 - Cache the selection state in nsTextFrame rather than in nsDisplayText r=emilio,jfkthame Differential Revision: https://phabricator.services.mozilla.com/D108646
5ca4929169fa74ec9def22bd995081acea3f95b8: Bug 1700677 - set layout.css.font-metrics-overrides.enabled to let css-fonts test pass with Beta and Release. r=jfkthame DONTBUILD
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 25 Mar 2021 17:57:42 +0000 - rev 573120
Push 38321 by smolnar@mozilla.com at Fri, 26 Mar 2021 04:08:09 +0000
Bug 1700677 - set layout.css.font-metrics-overrides.enabled to let css-fonts test pass with Beta and Release. r=jfkthame DONTBUILD They are slated to ride the trains starting with this cycle (Gecko 89). @font-face metrics override is only enabled in Nightly and controlled by the preference layout.css.font-metrics-overrides.enabled (added in bug 1681691). Differential Revision: https://phabricator.services.mozilla.com/D109651
570a3dbde1a51d5960b5d909c9248a22b8019ee2: Bug 708901 - Migrate to nsTHashSet in gfx/thebes. r=jfkthame,jrmuizel
Simon Giesecke <sgiesecke@mozilla.com> - Tue, 23 Mar 2021 10:36:36 +0000 - rev 572584
Push 38313 by apavel@mozilla.com at Tue, 23 Mar 2021 21:43:59 +0000
Bug 708901 - Migrate to nsTHashSet in gfx/thebes. r=jfkthame,jrmuizel Depends on D108602 Differential Revision: https://phabricator.services.mozilla.com/D108603
0b056792a101bd845bceb47b5a5aea81e7166a49: Bug 1697666 - Move IsSingleFaceFamily into gfxFontFamily. r=jfkthame
Tooru Fujisawa <arai_a@mac.com> - Wed, 17 Mar 2021 19:31:12 +0000 - rev 571668
Push 38295 by btara@mozilla.com at Thu, 18 Mar 2021 04:15:43 +0000
Bug 1697666 - Move IsSingleFaceFamily into gfxFontFamily. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D108810
bfdc3d20b85aba7240f595c184d12b1faec3d821: Bug 1697076 - Drop assertion from mozilla::unicode::GetNaked. r=jfkthame
Alex Henrie <alexhenrie24@gmail.com> - Thu, 11 Mar 2021 09:42:18 +0000 - rev 570651
Push 38278 by ncsoregi@mozilla.com at Thu, 11 Mar 2021 22:00:18 +0000
Bug 1697076 - Drop assertion from mozilla::unicode::GetNaked. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D107942
0643f14fa8fc76d300a1520306cb1d33d12a4586: Bug 1696935 - Update OTS to 8.1.4. r=jfkthame
Ryan VanderMeulen <ryanvm@gmail.com> - Wed, 10 Mar 2021 15:45:45 +0000 - rev 570500
Push 38276 by dluca@mozilla.com at Wed, 10 Mar 2021 21:58:46 +0000
Bug 1696935 - Update OTS to 8.1.4. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D107493
d5a14597ca06ee83747a62b85a33010316a53da1: Bug 1696940: Use nsRuleBreaker code in nsUniscribeBreaker when win32k lockdown is enabled. r=jfkthame
Bob Owen <bobowencode@gmail.com> - Wed, 10 Mar 2021 15:45:41 +0000 - rev 570499
Push 38276 by dluca@mozilla.com at Wed, 10 Mar 2021 21:58:46 +0000
Bug 1696940: Use nsRuleBreaker code in nsUniscribeBreaker when win32k lockdown is enabled. r=jfkthame This is only intended for testing, because win32k lockdown is disabled by default for content. Differential Revision: https://phabricator.services.mozilla.com/D107495
5355dd8298267f07f5f72b8f9d89e86255b225c0: Bug 1696407: Don't use GDI font table loading for DWrite when win32k lockdown is enabled. r=jfkthame
Bob Owen <bobowencode@gmail.com> - Wed, 10 Mar 2021 14:26:53 +0000 - rev 570488
Push 38276 by dluca@mozilla.com at Wed, 10 Mar 2021 21:58:46 +0000
Bug 1696407: Don't use GDI font table loading for DWrite when win32k lockdown is enabled. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D107488
0c69d6d5aab76328c9992a0febeb109a75c808cf: Bug 1649187 - Fix diacritic stripping for characters outside the BMP. r=jfkthame
Alex Henrie <alexhenrie24@gmail.com> - Wed, 10 Mar 2021 12:08:49 +0000 - rev 570454
Push 38276 by dluca@mozilla.com at Wed, 10 Mar 2021 21:58:46 +0000
Bug 1649187 - Fix diacritic stripping for characters outside the BMP. r=jfkthame Due to an unfortunate typo I made in base_chars.py, I thought that there were no mappings we care about outside of the basic multilingual plane. This patch adds back the non-BMP mappings that we do care about. Differential Revision: https://phabricator.services.mozilla.com/D107404
d61a3c845eb6b4c8ab9fae03a8d9b3eb4ca2806f: Bug 1649187 - Use a fallback table to strip diacritics from non-decomposable characters. r=jfkthame
Alex Henrie <alexhenrie24@gmail.com> - Sun, 07 Mar 2021 16:17:41 +0000 - rev 569925
Push 38269 by btara@mozilla.com at Sun, 07 Mar 2021 21:34:30 +0000
Bug 1649187 - Use a fallback table to strip diacritics from non-decomposable characters. r=jfkthame Implement the design suggested at https://bugzilla.mozilla.org/show_bug.cgi?id=1652910#c5 Differential Revision: https://phabricator.services.mozilla.com/D106674
7c8a779413425c81ebee3c49018d048f88758dfc: Bug 1692809 - Move some word-break tests into WPT. r=jfkthame
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Mon, 01 Mar 2021 16:55:55 +0000 - rev 569101
Push 38252 by ncsoregi@mozilla.com at Mon, 01 Mar 2021 22:00:15 +0000
Bug 1692809 - Move some word-break tests into WPT. r=jfkthame Actually, WPT has no some word-break tests for Japanese (ID and CJ) and Korean (Hangul Jamos). So I would like to move layout/reftests/text/wordbreak-2.html and layout/reftests/text/wordbreak-4a.html into WPT. Differential Revision: https://phabricator.services.mozilla.com/D105227
bd94eb676ba36f91b6192109a1dc8e3e1d4d4ad9: Bug 1693541 - Improve uses of nsBaseHashtable and descendants and avoid multiple subsequent lookups in intl/locale. r=jfkthame
Simon Giesecke <sgiesecke@mozilla.com> - Fri, 26 Feb 2021 11:36:39 +0000 - rev 568896
Push 38244 by abutkovits@mozilla.com at Fri, 26 Feb 2021 21:45:01 +0000
Bug 1693541 - Improve uses of nsBaseHashtable and descendants and avoid multiple subsequent lookups in intl/locale. r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D106117
58f29cf068ae1c4d3195358438b4b674efb54232: Bug 1692220 - Add a test to ensure the com.apple.FontRegistry dir is readable from content processes r=jfkthame
Haik Aftandilian <haftandilian@mozilla.com> - Fri, 19 Feb 2021 21:57:00 +0000 - rev 568141
Push 38221 by cbrindusan@mozilla.com at Sat, 20 Feb 2021 09:33:16 +0000
Bug 1692220 - Add a test to ensure the com.apple.FontRegistry dir is readable from content processes r=jfkthame Differential Revision: https://phabricator.services.mozilla.com/D105822
7bdc9784d1aa7c089dacee29fe1001cceb1f9103: Bug 1693395 - Add XROutputColor:PrintAsGrayscale to the grayscale values for CUPS. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 18 Feb 2021 05:18:38 +0000 - rev 567926
Push 38214 by smolnar@mozilla.com at Thu, 18 Feb 2021 09:24:11 +0000
Bug 1693395 - Add XROutputColor:PrintAsGrayscale to the grayscale values for CUPS. r=jfkthame See bug 1676191 comment 32. Differential Revision: https://phabricator.services.mozilla.com/D105513
6263c7f0613c9e9a849ca7cd509cbb94580c4d06: Bug 1692316 - Part 3: Document Duplex Printing Constants r=dholbert,jfkthame
Erik Nordin <nordzilla@mozilla.com> - Wed, 17 Feb 2021 23:57:27 +0000 - rev 567913
Push 38214 by smolnar@mozilla.com at Thu, 18 Feb 2021 09:24:11 +0000
Bug 1692316 - Part 3: Document Duplex Printing Constants r=dholbert,jfkthame This patch attempts to add a clear explanation and mapping among the duplex-printing constants that vary across platforms. Differential Revision: https://phabricator.services.mozilla.com/D105512
ceaebebccb5e67929ed7ba66d60954a9f218dc26: Bug 1691875 Part 3 - Always compute inner table's border and padding before initializing its ReflowInput. r=layout-reviewers,jfkthame,mats
Ting-Yu Lin <tlin@mozilla.com> - Mon, 15 Feb 2021 18:30:03 +0000 - rev 567564
Push 38206 by btara@mozilla.com at Tue, 16 Feb 2021 03:10:51 +0000
Bug 1691875 Part 3 - Always compute inner table's border and padding before initializing its ReflowInput. r=layout-reviewers,jfkthame,mats This is a preparation patch for bug 1674302. We'll need border padding to reduce the size for inner table size overrides. Differential Revision: https://phabricator.services.mozilla.com/D104643
3fabd86d59e6a6d473fa8f216f6715eb8065131c: Bug 1691875 Part 2 - Pass the area occupied by caption into CreateReflowInputForInnerTable(). r=mats,jfkthame
Ting-Yu Lin <tlin@mozilla.com> - Mon, 15 Feb 2021 18:30:03 +0000 - rev 567563
Push 38206 by btara@mozilla.com at Tue, 16 Feb 2021 03:10:51 +0000
Bug 1691875 Part 2 - Pass the area occupied by caption into CreateReflowInputForInnerTable(). r=mats,jfkthame If the table wrapper frame has a caption, several sizes that are used to initialized inner table's ReflowInput needs to be reduced: 1. the available block-size, 2. containing block size, 3. StyleSizeOverrides (will be considered in bug 1674302). This patch gives CreateReflowInputForInnerTable() an argument to pass the space occupied by the caption in order to reduce the above sizes. Note the old code doesn't reduce the constrained available block-size when recreating ReflowInput for a table grid item, but this patch does. Also, this patch goes ahead to recreate ReflowInput when the area occupied by caption is non-empty, which may or may not affect the containing block size for grid item. But for now, I keep it simple because bug 16743202 is going to add logic that needs to recreate ReflowInput when there is any size overrides. I tend not to over-optimize the performance. Differential Revision: https://phabricator.services.mozilla.com/D104642
06357a2a1fd35f0c5087780416adf982d61a6270: Bug 1691875 Part 1 - Split and rename nsTableWrapperFrame::ChildShrinkWrapISize() into two methods for caption and inner table. r=layout-reviewers,jfkthame
Ting-Yu Lin <tlin@mozilla.com> - Mon, 15 Feb 2021 18:30:02 +0000 - rev 567562
Push 38206 by btara@mozilla.com at Tue, 16 Feb 2021 03:10:51 +0000
Bug 1691875 Part 1 - Split and rename nsTableWrapperFrame::ChildShrinkWrapISize() into two methods for caption and inner table. r=layout-reviewers,jfkthame This patch is similar to Bug 1690901 Part 2, but for ChildShrinkWrapISize(). https://hg.mozilla.org/mozilla-central/rev/33171dca5ccd Also, aCBSize should be passed by const-reference, not by value. Differential Revision: https://phabricator.services.mozilla.com/D104641
aa4c7f4629ff48249dd37711fd07523d48778d00: Bug 1691256 - Don't reflow for background/border changes on xul appearance values. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 15 Feb 2021 13:19:32 +0000 - rev 567525
Push 38206 by btara@mozilla.com at Tue, 16 Feb 2021 03:10:51 +0000
Bug 1691256 - Don't reflow for background/border changes on xul appearance values. r=jfkthame This is an optimization, but also papers about XUL widgets not dealing particularly well with it. Differential Revision: https://phabricator.services.mozilla.com/D105169