searching for reviewer(heycam)
8a7ab9b279fe: Bug 1519501 - Generalize a bit GenerateServoStyleConsts and rename to RunCbindgen. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 13 Jan 2019 00:42:48 +0100 - rev 453707
Push 111131 by emilio@crisal.io at Mon, 14 Jan 2019 12:45:32 +0000
Bug 1519501 - Generalize a bit GenerateServoStyleConsts and rename to RunCbindgen. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D16399
472bcde2c2d1: Bug 1518954 - Three-value position syntax uses calc() as its computed value representation. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 11 Jan 2019 01:07:22 +0100 - rev 453291
Push 111081 by emilio@crisal.io at Fri, 11 Jan 2019 00:19:31 +0000
Bug 1518954 - Three-value position syntax uses calc() as its computed value representation. r=heycam This restores the previous behavior of using calc(). Note that background-position / object-position, which test this, weren't hitting the assertion because they use another codepath. I didn't add more extensive tests for this because it's well tested for those two properties, and because this is legacy anyway, see the comment in the test. I did add the assertion to the codepath those two properties hit. Differential Revision: https://phabricator.services.mozilla.com/D16176
3131c1c9f3c5: Bug 1518263 - Use Intl.NumberFormat objects to format numbers in about:memory rather than hand-rolling formatting code or using toLocaleString that will create a fresh formatter object every time it's called. r=heycam
Jeff Walden <jwalden@mit.edu> - Mon, 07 Jan 2019 12:05:55 -0800 - rev 452792
Push 110980 by rgurzau@mozilla.com at Tue, 08 Jan 2019 04:51:14 +0000
Bug 1518263 - Use Intl.NumberFormat objects to format numbers in about:memory rather than hand-rolling formatting code or using toLocaleString that will create a fresh formatter object every time it's called. r=heycam
624782f944fc: Bug 1517511 - Simplify computed::LengthOrPercentage and friends. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 07 Jan 2019 11:00:27 +0000 - rev 452701
Push 110957 by cbrindusan@mozilla.com at Mon, 07 Jan 2019 16:50:15 +0000
Bug 1517511 - Simplify computed::LengthOrPercentage and friends. r=heycam This is a first step to share LengthOrPercentage representation between Rust and Gecko. We need to preserve whether the value came from a calc() expression, for now at least, since we do different things depending on whether we're calc or not right now. See https://github.com/w3c/csswg-drafts/issues/3482 and dependent bugs for example. That means that the gecko conversion code needs to handle calc() in a bit of an awkward way until I change it to not be needed (patches for that incoming in the next few weeks I hope). I need to add a hack to exclude other things from the PartialEq implementation because the new conversion code is less lossy than the old one, and we relied on the lousiness in AnimationValue comparison (in order to start transitions and such, in [1] for example). I expect to remove that manual PartialEq implementation as soon as I'm done with the conversion. The less lossy conversion does fix a few serialization bugs for animation values though, like not loosing 0% values in calc() when interpolating lengths and percentages, see the two modified tests: * property-types.js * test_animation_properties.html Differential Revision: https://phabricator.services.mozilla.com/D15793
348ab7a8ab4d: Bug 1517682 - Drop layout.css.background-blend-mode.enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:21:32 +0000 - rev 452536
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.background-blend-mode.enabled pref. r=heycam It was enabled by default in bug 970600. Depends on D15707 Differential Revision: https://phabricator.services.mozilla.com/D15708
88046e1983f9: Bug 1517682 - Drop layout.css.scroll-behavior.property-enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:21:53 +0000 - rev 452535
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.scroll-behavior.property-enabled pref. r=heycam It was enabled by default in bug 1041833 (for desktops) and bug 1087562 (for Fennect). Depends on D15706 Differential Revision: https://phabricator.services.mozilla.com/D15707
f0fd2ca467ef: Bug 1517682 - Drop layout.css.mix-blend-mode.enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:22:13 +0000 - rev 452534
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.mix-blend-mode.enabled pref. r=heycam It was enabled by default in bug 952643. Depends on D15705 Differential Revision: https://phabricator.services.mozilla.com/D15706
8f2b23321d9a: Bug 1517682 - Drop layout.css.isolation.enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:22:27 +0000 - rev 452533
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.isolation.enabled pref. r=heycam It was enabled by default in bug 1091885. Depends on D15704 Differential Revision: https://phabricator.services.mozilla.com/D15705
3e992b70fb1b: Bug 1517682 - Drop layout.css.image-orientation.enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:22:52 +0000 - rev 452532
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.image-orientation.enabled pref. r=heycam It was enabled by default in bug 825771. Depends on D15703 Differential Revision: https://phabricator.services.mozilla.com/D15704
a8a86cdb127b: Bug 1517682 - Drop layout.css.color-adjust.enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:23:38 +0000 - rev 452531
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.color-adjust.enabled pref. r=heycam It was enabled by default in bug 1209273. Depends on D15702 Differential Revision: https://phabricator.services.mozilla.com/D15703
5a48406a5805: Bug 1517682 - Drop layout.css.box-decoration-break.enabled pref. r=heycam
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Fri, 04 Jan 2019 07:23:07 +0000 - rev 452530
Push 110915 by csabou@mozilla.com at Fri, 04 Jan 2019 16:47:57 +0000
Bug 1517682 - Drop layout.css.box-decoration-break.enabled pref. r=heycam It was enabled by default in bug 1006326. Differential Revision: https://phabricator.services.mozilla.com/D15702
8ad209d77614: Bug 1515556 comment fix-ups for the tree reformat (layout/) r=Ehsan,heycam
Sylvestre Ledru <sledru@mozilla.com> and Karl Tomlinson <karlt+@karlt.net> - Fri, 21 Dec 2018 03:07:05 +0000 - rev 451646
Push 110700 by ebalazs@mozilla.com at Fri, 21 Dec 2018 09:45:20 +0000
Bug 1515556 comment fix-ups for the tree reformat (layout/) r=Ehsan,heycam Differential Revision: https://phabricator.services.mozilla.com/D13125
1b1eaa3248a7: Bug 1513920 - Make <use> shadow trees lookup keyframe rules in the containing tree. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 17 Dec 2018 13:14:19 +0000 - rev 450947
Push 110585 by aiakab@mozilla.com at Mon, 17 Dec 2018 22:06:08 +0000
Bug 1513920 - Make <use> shadow trees lookup keyframe rules in the containing tree. r=heycam The same thing we do for rule matching. Differential Revision: https://phabricator.services.mozilla.com/D14548
7b04cdcfdcf4: Bug 1512026 - Handle nested slots correctly in slotted matching and invalidation. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 13 Dec 2018 02:17:53 +0000 - rev 450409
Push 110504 by btara@mozilla.com at Thu, 13 Dec 2018 22:55:26 +0000
Bug 1512026 - Handle nested slots correctly in slotted matching and invalidation. r=heycam The patch and test should be pretty much self-descriptive. Differential Revision: https://phabricator.services.mozilla.com/D14063
8bf181f9b1c3: Bug 1513387: Add braces & newlines to re-wrap some return statements that clang-format unwrapped, in layout/svg. r=heycam
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 12 Dec 2018 06:32:44 +0000 - rev 450246
Push 110462 by rmaries@mozilla.com at Wed, 12 Dec 2018 16:38:27 +0000
Bug 1513387: Add braces & newlines to re-wrap some return statements that clang-format unwrapped, in layout/svg. r=heycam This patch shouldn't affect behavior; it's just making the existing control flow clearer and more debuggable. Differential Revision: https://phabricator.services.mozilla.com/D14216
736e6ca6a7e1: Bug 1513012 - Move overflow to use cbindgen. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 11 Dec 2018 01:50:32 +0100 - rev 449903
Push 110428 by nbeleuzu@mozilla.com at Tue, 11 Dec 2018 05:26:22 +0000
Bug 1513012 - Move overflow to use cbindgen. r=heycam It's one of the most annoying / hacky mako bits we have. Differential Revision: https://phabricator.services.mozilla.com/D14084
49cf198748e0: Bug 1513058 - Deduplicate a bit the malloc_size_of code for hashmaps / hashsets. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 11 Dec 2018 00:11:18 +0000 - rev 449898
Push 110428 by nbeleuzu@mozilla.com at Tue, 11 Dec 2018 05:26:22 +0000
Bug 1513058 - Deduplicate a bit the malloc_size_of code for hashmaps / hashsets. r=heycam This allows to experiment with other hash maps easily rather than depending on what hashglobe::fake::HashMap dereferences to. In particular I wrote it while trying to get a build working with hashbrown. Differential Revision: https://phabricator.services.mozilla.com/D14098
6fd0a7cc9a43: Bug 1511730 patch 3 - Verify there are no frame state bit conflicts within groups. r=heycam
L. David Baron <dbaron@dbaron.org> - Wed, 05 Dec 2018 15:44:01 +0000 - rev 449442
Push 110371 by shindli@mozilla.com at Wed, 05 Dec 2018 21:41:36 +0000
Bug 1511730 patch 3 - Verify there are no frame state bit conflicts within groups. r=heycam Without patch 1, this correctly triggers: Assertion failure: !(bitsUsedPerGroup[group] & bit) (BULLET_FRAME_HAS_FONT_INFLATION must not use a bit already declared within its group), at /home/dbaron/builds/mozilla-central/mozilla/layout/generic/nsFrameStateBits.h:594 Differential Revision: https://phabricator.services.mozilla.com/D13779
6faca87a7549: Bug 1511730 patch 2 - Refactor the frame state group macros so that there's a unique header for every group. r=heycam
L. David Baron <dbaron@dbaron.org> - Wed, 05 Dec 2018 14:27:37 +0000 - rev 449441
Push 110371 by shindli@mozilla.com at Wed, 05 Dec 2018 21:41:36 +0000
Bug 1511730 patch 2 - Refactor the frame state group macros so that there's a unique header for every group. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D13778
3984125e9c65: Bug 1511730 patch 1 - But nsBulletFrame-specific state bits in the correct group. r=heycam
L. David Baron <dbaron@dbaron.org> - Wed, 05 Dec 2018 14:27:29 +0000 - rev 449440
Push 110371 by shindli@mozilla.com at Wed, 05 Dec 2018 21:41:36 +0000
Bug 1511730 patch 1 - But nsBulletFrame-specific state bits in the correct group. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D13777
c5b713000513: Bug 1509717 - Use cbindgen for border-style and outline-style. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 30 Nov 2018 05:27:28 +0000 - rev 448944
Push 110268 by nerli@mozilla.com at Fri, 30 Nov 2018 09:52:43 +0000
Bug 1509717 - Use cbindgen for border-style and outline-style. r=heycam I'm pretty sure the FIXME I left in the outline-style code is a bug, but I want to clean this up further and I didn't want to fix it without adding a test. Differential Revision: https://phabricator.services.mozilla.com/D12859
70030bab1073: Bug 1509717 - Move BorderStyle to border.rs. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 30 Nov 2018 05:16:52 +0000 - rev 448943
Push 110268 by nerli@mozilla.com at Fri, 30 Nov 2018 09:52:43 +0000
Bug 1509717 - Move BorderStyle to border.rs. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D12858
b4c0c0fecdba: Bug 775618 - Implement page-break-{before,after} as legacy shorthands for {before,after}. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Fri, 30 Nov 2018 05:35:47 +0000 - rev 448941
Push 110268 by nerli@mozilla.com at Fri, 30 Nov 2018 09:52:43 +0000
Bug 775618 - Implement page-break-{before,after} as legacy shorthands for {before,after}. r=heycam This is all the style-system work needed for this. This implements the concept of legacy shorthands, teaches tests to understand it, and adds a few more tests for these properties in particular. The WPT even caught a few WebKit / Blink bugs: https://bugs.chromium.org/p/chromium/issues/detail?id=906336 https://bugs.webkit.org/show_bug.cgi?id=191803 This doesn't change the layout behavior for page-break-before: always, since it'd stop breaking in multicol and such. Similarly, break-before / break-after: column and page still behave the same, I'll file followups for those given comment 22. Differential Revision: https://phabricator.services.mozilla.com/D12211
f97f45821194: Bug 775618 - Introduce the concept of legacy shorthands. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 20 Nov 2018 04:39:36 +0000 - rev 448940
Push 110268 by nerli@mozilla.com at Fri, 30 Nov 2018 09:52:43 +0000
Bug 775618 - Introduce the concept of legacy shorthands. r=heycam We need this because there's a weird mapping between these properties' values ('always' maps to 'page'). See https://drafts.csswg.org/css-cascade-4/#legacy-shorthand. Differential Revision: https://phabricator.services.mozilla.com/D12213
c228095e647e: Bug 775618 - Introduce break-inside, and alias page-break-inside to it. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 20 Nov 2018 05:07:52 +0000 - rev 448939
Push 110268 by nerli@mozilla.com at Fri, 30 Nov 2018 09:52:43 +0000
Bug 775618 - Introduce break-inside, and alias page-break-inside to it. r=heycam This property has no weird value mapping, so we can just do this. Differential Revision: https://phabricator.services.mozilla.com/D12212
b5e141f4ca9b: Bug 1509664 - Change Text Rendering constants to enum classes in nsStyleConsts.h. r=heycam
Shubham Kumaram <shubhamkumaram@gmail.com> - Fri, 30 Nov 2018 05:08:46 +0000 - rev 448925
Push 110268 by nerli@mozilla.com at Fri, 30 Nov 2018 09:52:43 +0000
Bug 1509664 - Change Text Rendering constants to enum classes in nsStyleConsts.h. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D12839
923646987d58: Bug 1504918. Prevent TextNodeCorrespondenceRecorder::TraverseAndRecord crash. r=heycam
Jonathan Watt <jwatt@jwatt.org> - Tue, 13 Nov 2018 16:49:21 +0000 - rev 448224
Push 110149 by jwatt@jwatt.org at Tue, 27 Nov 2018 15:58:33 +0000
Bug 1504918. Prevent TextNodeCorrespondenceRecorder::TraverseAndRecord crash. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D11107
76aa81358d76: Bug 1507305 - Use Servo to serialize most of the already-exposed shorthands. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 04:10:08 +0000 - rev 446571
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1507305 - Use Servo to serialize most of the already-exposed shorthands. r=heycam Skip mask and text-decoration for now since there's a single test-case failing for each that seem worth fixing in a different bug: * For mask, there's the case of setting mask: url(foo.svg#bar), which we test we serialize absolutely. But given we're uncomputing it we don't serialize the resolved URL. Chrome doesn't either so we could just change the test, but even if we decided to do it we probably should do it in a separate bug. * For text-decoration, we need to resolve the value, when it's an interpolation between currentcolor and other color. Right now that returns the empty string which is not great: https://searchfox.org/mozilla-central/rev/d850d799a0009f851b5535580e0a8b4bb2c591d7/servo/components/style/values/specified/color.rs#194 So I need to come up with something. Probably we need to implement the "hard" version of the serialization code that doesn't reuse the animation machinery. Definitely a separate bug though. While at it, also serialize all <position> longhands with Servo, so that I can clean up the tests. Differential Revision: https://phabricator.services.mozilla.com/D11948
5201cfe20675: Bug 1507305 - Add a mechanism to serialize shorthands for getComputedStyle(). r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 08:25:13 +0000 - rev 446570
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1507305 - Add a mechanism to serialize shorthands for getComputedStyle(). r=heycam This implements the mechanism reusing the animation machinery for now, so it asserts in a few cases that this wouldn't handle correctly. For shorthands that have colors and other bits we'd need a more sophisticated mechanism with a bit more code (that resolves colors and such), but it'd look something like this regardless, and we should have this in any case. Differential Revision: https://phabricator.services.mozilla.com/D11944
6c54e79a66b5: Bug 1507311 - Make the serialization of the mask shorthand more compact. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 08:22:31 +0000 - rev 446569
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1507311 - Make the serialization of the mask shorthand more compact. r=heycam This helps to preserve the old longhand form when possible (mask used to be a longhand), which will be relevant when we serialize this for the computed value. Differential Revision: https://phabricator.services.mozilla.com/D11943
8ee9454879c8: Bug 1507309 - Simplify background-repeat. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 03:23:39 +0000 - rev 446568
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1507309 - Simplify background-repeat. r=heycam This way we always serialize in the shortest form, and take less space. This is useful because when serializing uncomputed values we'd like to compare to the initial value to avoid serializing parts of a shorthand, but with the existing implementation we would generate always a second keyword, which means that we'll never match it. This also matches Chrome and WebKit, incidentally, so I'm pretty confident the behavior change when serializing specified style is web-compatible. Differential Revision: https://phabricator.services.mozilla.com/D11941
488a0a90ca91: Bug 1507127 - Also move page-break-inside outside of mako. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 08:16:23 +0000 - rev 446567
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1507127 - Also move page-break-inside outside of mako. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D11876
f678d3408301: Bug 1507127 - Move the page-break-{before,after} properties to not use mako. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 08:15:13 +0000 - rev 446566
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1507127 - Move the page-break-{before,after} properties to not use mako. r=heycam And respect the computed value of `left` / `right` / etc. Differential Revision: https://phabricator.services.mozilla.com/D11872
ddd1c83fb4e8: Bug 1433439 - Don't expose non-content-accessible pseudo-elements from getComputedStyle. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Nov 2018 02:35:23 +0000 - rev 446565
Push 109893 by ebalazs@mozilla.com at Thu, 15 Nov 2018 16:22:51 +0000
Bug 1433439 - Don't expose non-content-accessible pseudo-elements from getComputedStyle. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D11830
26666002fbe3: Bug 1499961 - Indicate position-change reflow hint when RecomputePosition() bails out - r=dbaron,heycam
Gerald Squelart <gsquelart@mozilla.com> - Wed, 14 Nov 2018 10:16:01 +0000 - rev 446357
Push 109870 by rmaries@mozilla.com at Wed, 14 Nov 2018 22:38:51 +0000
Bug 1499961 - Indicate position-change reflow hint when RecomputePosition() bails out - r=dbaron,heycam If a frame cannot be repositioned, the reflow request should hint at the position change, so that if that frame is a reflow root, the next reflow won't just start from there and not set the new position. Updated incorrect comment about nsChangeHint_ReflowChangesSizeOrPosition always requiring nsChangeHint_ClearAncestorIntrinsics. Differential Revision: https://phabricator.services.mozilla.com/D9065
ed063368a801: Bug 1505949 - layout/style/{BorrowedTypeList.h,ServoArcTypeList.h,ServoBoxedTypeList.h} content should remain on a single line r=heycam
Sylvestre Ledru <sledru@mozilla.com> - Fri, 09 Nov 2018 13:31:39 +0000 - rev 445438
Push 109742 by dluca@mozilla.com at Fri, 09 Nov 2018 17:47:42 +0000
Bug 1505949 - layout/style/{BorrowedTypeList.h,ServoArcTypeList.h,ServoBoxedTypeList.h} content should remain on a single line r=heycam Make sure that clang-format doesn't touch it Differential Revision: https://phabricator.services.mozilla.com/D11387
81d1d93099bf: Bug 1505565 - Split up push_applicable_declarations. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Nov 2018 23:07:40 +0000 - rev 445343
Push 109718 by rgurzau@mozilla.com at Fri, 09 Nov 2018 05:49:16 +0000
Bug 1505565 - Split up push_applicable_declarations. r=heycam Introduce RuleCollector, which contains all the state we need during the cascade, and allows to reuse a bit of code. Differential Revision: https://phabricator.services.mozilla.com/D11233
9bdd38db48e6: Bug 1505816 - Cleanup text-align. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Nov 2018 22:57:18 +0000 - rev 445340
Push 109718 by rgurzau@mozilla.com at Fri, 09 Nov 2018 05:49:16 +0000
Bug 1505816 - Cleanup text-align. r=heycam Use derive(FromPrimitive) instead. Differential Revision: https://phabricator.services.mozilla.com/D11332
8735038d615f: Bug 1505213 - Allow ::slotted()::placeholder. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Nov 2018 13:33:59 +0000 - rev 445179
Push 109695 by rgurzau@mozilla.com at Thu, 08 Nov 2018 19:16:54 +0000
Bug 1505213 - Allow ::slotted()::placeholder. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D11132
a86fa4d83651: Bug 1505618 - Add Servo's rustfmt.toml to the servo/ directory. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 08 Nov 2018 00:42:19 +0100 - rev 445043
Push 109663 by emilio@crisal.io at Wed, 07 Nov 2018 23:50:45 +0000
Bug 1505618 - Add Servo's rustfmt.toml to the servo/ directory. r=heycam
6c4703b12156: Bug 1504034 - followup post-review nit. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 07 Nov 2018 00:18:29 +0100 - rev 444784
Push 109636 by ncsoregi@mozilla.com at Wed, 07 Nov 2018 10:00:15 +0000
Bug 1504034 - followup post-review nit. r=heycam
ead2368dc078: Bug 1479173 - Add a static constexpr function returns an nsCSSPropertyIDSet being consist of CSS properties set can be run on the compositor. r=heycam,birtles
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Tue, 06 Nov 2018 21:00:38 +0000 - rev 444743
Push 109636 by ncsoregi@mozilla.com at Wed, 07 Nov 2018 10:00:15 +0000
Bug 1479173 - Add a static constexpr function returns an nsCSSPropertyIDSet being consist of CSS properties set can be run on the compositor. r=heycam,birtles Also add a script to generate the CSS properties set by looking at CanAnimateOnCompositor flag in servo's property definitions. Differential Revision: https://phabricator.services.mozilla.com/D10687
7cf0ad1af067: Bug 1479173 - Add a static constexpr function returns an nsCSSPropertyIDSet being consist of CSS properties set can be run on the compositor. r=heycam,birtles
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Tue, 06 Nov 2018 09:38:57 +0000 - rev 444624
Push 109617 by apavel@mozilla.com at Tue, 06 Nov 2018 17:15:12 +0000
Bug 1479173 - Add a static constexpr function returns an nsCSSPropertyIDSet being consist of CSS properties set can be run on the compositor. r=heycam,birtles Also add a script to generate the CSS properties set by looking at CanAnimateOnCompositor flag in servo's property definitions. Differential Revision: https://phabricator.services.mozilla.com/D10687
15da260eaa3c: Bug 1462233 - Implement the env() function with hardcoded zeros for safe-area-inset. r=heycam,firefox-style-system-reviewers
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 05 Nov 2018 10:39:46 +0000 - rev 444397
Push 109582 by csabou@mozilla.com at Mon, 05 Nov 2018 16:21:20 +0000
Bug 1462233 - Implement the env() function with hardcoded zeros for safe-area-inset. r=heycam,firefox-style-system-reviewers Intent to Implement and Ship: https://groups.google.com/d/msg/mozilla.dev.platform/EVKyR1B87T0/_l-_qK8SAAAJ Differential Revision: https://phabricator.services.mozilla.com/D9609
6251012862a2: Bug 1504536 - Simplify the SVG animation code. r=hiro,heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 04 Nov 2018 15:48:08 +0100 - rev 444394
Push 109582 by csabou@mozilla.com at Mon, 05 Nov 2018 16:21:20 +0000
Bug 1504536 - Simplify the SVG animation code. r=hiro,heycam It's overly generic for no good reason. Differential Revision: https://phabricator.services.mozilla.com/D10844
dd629ef17f62: Bug 1502751 - Don't keep a separate list of ignored-when-colors-disabled longhands. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 05 Nov 2018 00:16:58 +0000 - rev 444386
Push 109582 by csabou@mozilla.com at Mon, 05 Nov 2018 16:21:20 +0000
Bug 1502751 - Don't keep a separate list of ignored-when-colors-disabled longhands. r=heycam Most of the change is moving sets around to be static functions on LonghandIdSet. I think I like that pattern, but I can also make the new set a global static and add mako code to be `pub` or something. Though I think the LonghandIdSet::foo().contains(..) pattern is nice to read :) Differential Revision: https://phabricator.services.mozilla.com/D10653
9f0e2fbf4d15: Bug 1502893 - Don't match document author rules if not needed for revalidation. r=heycam,firefox-style-system-reviewers
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 05 Nov 2018 00:05:12 +0000 - rev 444385
Push 109582 by csabou@mozilla.com at Mon, 05 Nov 2018 16:21:20 +0000
Bug 1502893 - Don't match document author rules if not needed for revalidation. r=heycam,firefox-style-system-reviewers When you're in a ShadowRoot and can share style with a sibling, the sharing code is smart enough to skip document author rules. But then it could get confused if you also include document rules, since revalidation selectors are matched against these. This is not a correctness issue, because we're matching more than what we need, and avoid sharing if we failed. Also fix the detection for user rules in any_applicable_rule_data. Differential Revision: https://phabricator.services.mozilla.com/D10117
b7ed6370bc7d: Bug 1502617 - Support ::before / ::after on ::slotted pseudos. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 29 Oct 2018 23:34:12 +0100 - rev 443423
Push 109366 by emilio@crisal.io at Mon, 29 Oct 2018 22:42:37 +0000
Bug 1502617 - Support ::before / ::after on ::slotted pseudos. r=heycam See https://github.com/w3c/csswg-drafts/issues/3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994
8ff29f3d64b7: Bug 1502936 - Fix SVGUseElement::GetFrame to not get confused. r=heycam
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 29 Oct 2018 20:30:22 +0100 - rev 443421
Push 109364 by emilio@crisal.io at Mon, 29 Oct 2018 22:27:27 +0000
Bug 1502936 - Fix SVGUseElement::GetFrame to not get confused. r=heycam Should've noticed in bug 1502658 that GetFrame() was dead, to verify its assumptions... Oh well. Differential Revision: https://phabricator.services.mozilla.com/D10109
9b13397e4ca5: Bug 1502510: Provide missing image file for calc-background-position and calc-background-size reftests. r=heycam
Daniel Holbert <dholbert@cs.stanford.edu> - Mon, 29 Oct 2018 00:57:16 +0000 - rev 443386
Push 109362 by rgurzau@mozilla.com at Mon, 29 Oct 2018 22:12:05 +0000
Bug 1502510: Provide missing image file for calc-background-position and calc-background-size reftests. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D9951