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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2403873002: Reformat remaining comments in core/layout (Closed)

Created:
4 years, 2 months ago by eae
Modified:
4 years, 2 months ago
Reviewers:
Nico
CC:
chromium-reviews, szager+layoutwatch_chromium.org, zoltan1, blink-reviews-layout_chromium.org, pdr+renderingwatchlist_chromium.org, eae+blinkwatch, leviw+renderwatch, jchaffraix+rendering, blink-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reformat remaining comments in core/layout BUG=563793 TBR=thakis@chromium.org Committed: https://crrev.com/b4f18350f219dd9b3b014ef86bffea9ea9028228 Cr-Commit-Position: refs/heads/master@{#424177}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M third_party/WebKit/Source/core/layout/LayoutText.h View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/ListMarkerText.h View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 7 (2 generated)
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/2403873002/1
4 years, 2 months ago (2016-10-10 15:52:13 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-10-10 17:28:55 UTC) #3
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/b4f18350f219dd9b3b014ef86bffea9ea9028228 Cr-Commit-Position: refs/heads/master@{#424177}
4 years, 2 months ago (2016-10-10 17:32:43 UTC) #5
Nico
lgtm, thanks! I had tried to reformat all of blink last Friday night. I wonder ...
4 years, 2 months ago (2016-10-10 17:39:32 UTC) #6
eae
4 years, 2 months ago (2016-10-10 17:55:22 UTC) #7
Message was sent while issue was closed. 
No worries at all. Might have gone in between the format and the CL that started
enforcing it.

Powered by Google App Engine
This is Rietveld 408576698