Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(117)

Issues Search
Open Issues | Closed Issues | All Issues | Sign in with your Google Account to create issues and add comments

Issue 1171853004: A whole lotta nits in core/layout/ (Closed)

Created:
4 years, 10 months ago by pilgrim_google
Modified:
4 years, 10 months ago
Reviewers:
eae
CC:
blink-reviews, krit, jfernandez, pdr+renderingwatchlist_chromium.org, zoltan1, svillar, Manuel Rego, szager+layoutwatch_chromium.org, eae+blinkwatch, leviw+renderwatch, kouhei+svg_chromium.org, fs, blink-reviews-rendering, f(malita), jchaffraix+rendering, gyuyoung2, Stephen Chennney, rwlbuis, pdr+svgwatchlist_chromium.org
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -25 lines) Patch
M Source/core/layout/LayoutBlock.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/layout/LayoutBox.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutBoxModelObject.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutFlexibleBox.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutGrid.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/layout/LayoutInline.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutObject.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/layout/LayoutObjectChildList.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutTable.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutTableCell.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutTableRow.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/LayoutTheme.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/layout/LayoutThemeTest.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/compositing/CompositedDeprecatedPaintLayerMapping.cpp View 3 chunks +3 lines, -2 lines 0 comments Download
M Source/core/layout/line/EllipsisBox.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/line/InlineFlowBox.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/svg/LayoutSVGBlock.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/svg/SVGResources.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/layout/svg/SVGTextLayoutEngineBaseline.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (1 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1171853004/1
4 years, 10 months ago (2015-06-10 20:37:01 UTC) #2
commit-bot: I haz the power
4 years, 10 months ago (2015-06-10 21:42:40 UTC) #3
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196900

Powered by Google App Engine
This is Rietveld 408576698