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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2405633002: Reformat comments in core/layout (Closed)

Created:
4 years, 2 months ago by eae
Modified:
4 years, 2 months ago
Reviewers:
pdr.
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 comments in core/layout BUG=563793 R=pdr@chromium.org Committed: https://crrev.com/bd1a920e6b9db5852c3248798723a70edfbf52e6 Cr-Commit-Position: refs/heads/master@{#424052}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+576 lines, -403 lines) Patch
M third_party/WebKit/Source/core/layout/LayoutTableRow.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTableSection.h View 8 chunks +22 lines, -13 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTableSection.cpp View 49 chunks +180 lines, -134 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTestHelper.cpp View 1 chunk +4 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutText.h View 2 chunks +13 lines, -11 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutText.cpp View 25 chunks +74 lines, -54 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextCombine.h View 1 chunk +3 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextCombine.cpp View 2 chunks +4 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextControl.h View 2 chunks +6 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextControl.cpp View 6 chunks +17 lines, -12 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextControlMultiLine.h View 2 chunks +4 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextControlMultiLine.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextControlSingleLine.h View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextControlSingleLine.cpp View 3 chunks +7 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextFragment.h View 3 chunks +11 lines, -9 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextFragment.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextTest.cpp View 4 chunks +8 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp View 1 chunk +4 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTheme.h View 3 chunks +32 lines, -24 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTheme.cpp View 9 chunks +28 lines, -17 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutThemeDefault.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutThemeMac.h View 3 chunks +6 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutThemeMac.mm View 3 chunks +8 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutThemeMobile.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTreeAsText.h View 3 chunks +7 lines, -5 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutTreeAsText.cpp View 3 chunks +7 lines, -4 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutVTTCue.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutVideo.cpp View 4 chunks +21 lines, -16 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutView.h View 5 chunks +15 lines, -9 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutView.cpp View 23 chunks +81 lines, -53 lines 0 comments Download

Messages

Total messages: 10 (5 generated)
eae
4 years, 2 months ago (2016-10-07 23:45:19 UTC) #1
pdr.
LGTM
4 years, 2 months ago (2016-10-08 00:00:53 UTC) #4
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/2405633002/1
4 years, 2 months ago (2016-10-08 00:14:23 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-10-08 02:49:50 UTC) #8
commit-bot: I haz the power
4 years, 2 months ago (2016-10-08 02:52:15 UTC) #10
Message was sent while issue was closed. 
Patchset 1 (id:??) landed as
https://crrev.com/bd1a920e6b9db5852c3248798723a70edfbf52e6
Cr-Commit-Position: refs/heads/master@{#424052}

Powered by Google App Engine
This is Rietveld 408576698