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

Issue 2878353002: Revert of Move CursorList to its own file. (Closed)

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

Description

Revert of Move CursorList to its own file. (patchset #2 id:20001 of https://codereview.chromium.org/2880963002/ ) Reason for revert: Reverting since the BUILD file wasn't updated. Author is aware Original issue's description: > Move CursorList to its own file. > > When we generate external types like Length in ComputedStyle, we specify > the path to the type (e.g. "platform/Length"). We then deduce the name > of the type from the path ("Length"). This means that the type must be > stored in a file with the same name. > > This patch moves CursorList to its own file so that it can be generated. > > BUG=628043 > > Review-Url: https://codereview.chromium.org/2880963002 > Cr-Commit-Position: refs/heads/master@{#471654} > Committed: https://chromium.googlesource.com/chromium/src/+/79e3cc6d695903af9f32b770fd80f67e878587e7 TBR=suzyh@chromium.org,shend@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=628043 Review-Url: https://codereview.chromium.org/2878353002 Cr-Commit-Position: refs/heads/master@{#471655} Committed: https://chromium.googlesource.com/chromium/src/+/9d45549ee7fdd99b2c8d536c1c18dd0e008dce5a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -18 lines) Patch
D third_party/WebKit/Source/core/style/CursorList.h View 1 chunk +0 lines, -17 lines 0 comments Download
M third_party/WebKit/Source/core/style/StyleRareInheritedData.h View 2 chunks +3 lines, -1 line 0 comments Download

Messages

Total messages: 7 (3 generated)
nainar
Created Revert of Move CursorList to its own file.
3 years, 7 months ago (2017-05-15 03:41:35 UTC) #2
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/2878353002/1
3 years, 7 months ago (2017-05-15 03:41:59 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/9d45549ee7fdd99b2c8d536c1c18dd0e008dce5a
3 years, 7 months ago (2017-05-15 03:42:32 UTC) #6
suzyh_UTC10 (ex-contributor)
3 years, 7 months ago (2017-05-15 04:32:03 UTC) #7
Message was sent while issue was closed.
lgtm

How did the patch land if it did not have the required BUILD changes?

Powered by Google App Engine
This is Rietveld 408576698