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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 700703002: Remove more float machinery. (Closed)

Created:
6 years, 1 month ago by ojan
Modified:
6 years, 1 month ago
Reviewers:
esprehn
CC:
esprehn, abarth-chromium, mojo-reviews_chromium.org
Base URL:
git@github.com:domokit/mojo.git@master
Project:
mojo
Visibility:
Public.

Description

Remove more float machinery. Also removed the clear property since it's only purpose was to clear floats. R=esprehn@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/7f8b8ac86411e4d54ac31a1a5ec816a07f238f4e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -338 lines) Patch
M sky/engine/core/css/CSSComputedStyleDeclaration.cpp View 3 chunks +0 lines, -8 lines 0 comments Download
M sky/engine/core/css/CSSPrimitiveValueMappings.h View 2 chunks +0 lines, -76 lines 0 comments Download
M sky/engine/core/css/CSSProperties.in View 2 chunks +0 lines, -2 lines 0 comments Download
M sky/engine/core/css/parser/BisonCSSParser-in.cpp View 3 chunks +0 lines, -6 lines 0 comments Download
M sky/engine/core/css/resolver/StyleAdjuster.cpp View 1 chunk +0 lines, -3 lines 0 comments Download
M sky/engine/core/editing/htmlediting.cpp View 1 chunk +0 lines, -7 lines 0 comments Download
M sky/engine/core/editing/markup.cpp View 6 chunks +7 lines, -16 lines 0 comments Download
M sky/engine/core/rendering/InlineIterator.h View 1 chunk +1 line, -1 line 0 comments Download
M sky/engine/core/rendering/RenderBlock.h View 2 chunks +0 lines, -3 lines 0 comments Download
M sky/engine/core/rendering/RenderBlock.cpp View 12 chunks +7 lines, -60 lines 0 comments Download
M sky/engine/core/rendering/RenderBlockFlow.h View 3 chunks +1 line, -5 lines 0 comments Download
M sky/engine/core/rendering/RenderBlockFlow.cpp View 6 chunks +4 lines, -22 lines 0 comments Download
M sky/engine/core/rendering/RenderBlockLineLayout.cpp View 7 chunks +10 lines, -34 lines 0 comments Download
M sky/engine/core/rendering/RenderBox.h View 2 chunks +0 lines, -8 lines 0 comments Download
M sky/engine/core/rendering/RenderBox.cpp View 7 chunks +3 lines, -27 lines 0 comments Download
M sky/engine/core/rendering/RenderLayerModelObject.cpp View 4 chunks +0 lines, -8 lines 0 comments Download
M sky/engine/core/rendering/RenderObject.h View 3 chunks +2 lines, -4 lines 0 comments Download
M sky/engine/core/rendering/RenderObject.cpp View 2 chunks +3 lines, -8 lines 0 comments Download
M sky/engine/core/rendering/line/BreakingContextInlineHeaders.h View 3 chunks +2 lines, -11 lines 0 comments Download
M sky/engine/core/rendering/line/LineBreaker.h View 2 chunks +0 lines, -2 lines 0 comments Download
M sky/engine/core/rendering/line/LineBreaker.cpp View 1 chunk +0 lines, -1 line 0 comments Download
M sky/engine/core/rendering/shapes/ShapeOutsideInfo.cpp View 1 chunk +1 line, -1 line 0 comments Download
M sky/engine/core/rendering/style/RenderStyle.h View 10 chunks +0 lines, -13 lines 0 comments Download
M sky/engine/core/rendering/style/RenderStyle.cpp View 2 chunks +0 lines, -4 lines 0 comments Download
M sky/engine/core/rendering/style/RenderStyleConstants.h View 2 chunks +0 lines, -8 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
ojan
6 years, 1 month ago (2014-11-04 00:45:57 UTC) #2
esprehn
lgtm
6 years, 1 month ago (2014-11-04 00:46:13 UTC) #4
ojan
6 years, 1 month ago (2014-11-04 01:00:32 UTC) #5
Message was sent while issue was closed. 
Committed patchset #1 (id:1) manually as
7f8b8ac86411e4d54ac31a1a5ec816a07f238f4e.

Powered by Google App Engine
This is Rietveld 408576698