Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Issue 2886233004: Generate diffing functions for generated subgroup InheritedData in ComputedStyle (Closed)

Created:
3 years, 7 months ago by nainar
Modified:
3 years, 7 months ago
CC:
darktears, apavlov+blink_chromium.org, blink-reviews, blink-reviews-css, blink-reviews-style_chromium.org, chromium-reviews, dglazkov+blink, rwlbuis
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Generate diffing functions for generated subgroup InheritedData in ComputedStyle This patch generates the diffing functions for StyleInheritedData which is now a generated subgroup of ComputedStyle. Please note: No added callsite for ComputedStyleBase::DiffNeedsFullLayoutAndPaintInvalidation has been added since it is already being called. We have just appended to that JSON entry. Diff: https://gist.github.com/nainar/5650d8c7dc6b9cc5d784c5be2409b338/revisions BUG=710938 Review-Url: https://codereview.chromium.org/2886233004 Cr-Commit-Position: refs/heads/master@{#473015} Committed: https://chromium.googlesource.com/chromium/src/+/702510f88b10b7ab5df0a87ada3d09f6d1139873

Patch Set 1 #

Total comments: 7
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -11 lines) Patch
M third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 View 1 chunk +1 line, -1 line 5 comments Download
M third_party/WebKit/Source/core/style/ComputedStyle.cpp View 1 chunk +0 lines, -10 lines 2 comments Download

Messages

Total messages: 25 (15 generated)
nainar
shend@, PTAL? Thanks!
3 years, 7 months ago (2017-05-18 08:28:12 UTC) #6
shend
The generated diff seems to be for another patch, can you reupload? https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 File third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 ...
3 years, 7 months ago (2017-05-18 22:17:51 UTC) #7
nainar
Updated the diff in the CL description https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 File third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 (right): https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5#newcode30 third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5:30: "padding-bottom", "line-height", ...
3 years, 7 months ago (2017-05-19 00:00:38 UTC) #9
shend
lgtm https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 File third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 (right): https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5#newcode30 third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5:30: "padding-bottom", "line-height", "font", "-webkit-border-horizontal-spacing", "-webkit-border-vertical-spacing"], On 2017/05/19 at ...
3 years, 7 months ago (2017-05-19 00:09:18 UTC) #10
nainar
meade, PTAL? Thanks! https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 File third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 (right): https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5#newcode30 third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5:30: "padding-bottom", "line-height", "font", "-webkit-border-horizontal-spacing", "-webkit-border-vertical-spacing"], On ...
3 years, 7 months ago (2017-05-19 00:15:02 UTC) #14
nainar
Subbing meade@ for alancutter@, PTAL? Thanks!
3 years, 7 months ago (2017-05-19 01:00:41 UTC) #18
alancutter (OOO until 2018)
lgtm https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 File third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 (right): https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5#newcode30 third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5:30: "padding-bottom", "line-height", "font", "-webkit-border-horizontal-spacing", "-webkit-border-vertical-spacing"], On 2017/05/19 at ...
3 years, 7 months ago (2017-05-19 01:16:58 UTC) #19
nainar
https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/style/ComputedStyle.cpp File third_party/WebKit/Source/core/style/ComputedStyle.cpp (left): https://codereview.chromium.org/2886233004/diff/1/third_party/WebKit/Source/core/style/ComputedStyle.cpp#oldcode734 third_party/WebKit/Source/core/style/ComputedStyle.cpp:734: } On 2017/05/19 at 01:16:58, alancutter wrote: > Haha, ...
3 years, 7 months ago (2017-05-19 01:18:20 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2886233004/1
3 years, 7 months ago (2017-05-19 01:19:49 UTC) #22
commit-bot: I haz the power
3 years, 7 months ago (2017-05-19 01:26:18 UTC) #25
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/702510f88b10b7ab5df0a87ada3d...

Powered by Google App Engine
This is Rietveld 408576698