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

Issue 363343005: Remove support for anonymous deprecated flexboxes. (Closed)

Created:
6 years, 5 months ago by harpreet.sk
Modified:
6 years, 4 months ago
CC:
blink-reviews, blink-reviews-rendering, eae+blinkwatch, jchaffraix+rendering, leviw+renderwatch, pdr., rune+blink, zoltan1
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

Remove support for anonymous deprecated flexboxes. There are no remaining internal clients of RenderDeprecatedFlexibleBox, so removing support for anonymous deprecated flexboxes. These changes has been taken from following WebKit patch http://trac.webkit.org/changeset/155689 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179655

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -22 lines) Patch
M Source/core/rendering/RenderBlock.cpp View 1 1 chunk +1 line, -5 lines 0 comments Download
M Source/core/rendering/RenderDeprecatedFlexibleBox.h View 1 3 chunks +4 lines, -7 lines 0 comments Download
M Source/core/rendering/RenderDeprecatedFlexibleBox.cpp View 1 2 chunks +2 lines, -9 lines 0 comments Download
M Source/core/rendering/RenderObject.cpp View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 12 (0 generated)
harpreet.sk
ptal..
6 years, 5 months ago (2014-07-07 14:50:51 UTC) #1
tony
cbiesinger would be a good reviewer for this.
6 years, 5 months ago (2014-07-07 16:59:38 UTC) #2
harpreet.sk
@Christian can you please review the change...
6 years, 5 months ago (2014-07-08 17:10:06 UTC) #3
cbiesinger
https://codereview.chromium.org/363343005/diff/1/Source/core/rendering/RenderDeprecatedFlexibleBox.cpp File Source/core/rendering/RenderDeprecatedFlexibleBox.cpp (left): https://codereview.chromium.org/363343005/diff/1/Source/core/rendering/RenderDeprecatedFlexibleBox.cpp#oldcode126 Source/core/rendering/RenderDeprecatedFlexibleBox.cpp:126: if (!isAnonymous()) { This entire if block needs to ...
6 years, 4 months ago (2014-08-06 03:00:30 UTC) #4
harpreet.sk
Incorporated the changes asked. Please take a look. https://codereview.chromium.org/363343005/diff/1/Source/core/rendering/RenderDeprecatedFlexibleBox.cpp File Source/core/rendering/RenderDeprecatedFlexibleBox.cpp (left): https://codereview.chromium.org/363343005/diff/1/Source/core/rendering/RenderDeprecatedFlexibleBox.cpp#oldcode126 Source/core/rendering/RenderDeprecatedFlexibleBox.cpp:126: if ...
6 years, 4 months ago (2014-08-06 13:22:14 UTC) #5
cbiesinger
lgtm
6 years, 4 months ago (2014-08-06 13:51:30 UTC) #6
harpreet.sk
@leviw: Requires owner's lgtm
6 years, 4 months ago (2014-08-06 14:25:01 UTC) #7
leviw_travelin_and_unemployed
lgtm
6 years, 4 months ago (2014-08-06 18:11:29 UTC) #8
harpreet.sk
The CQ bit was checked by harpreet.sk@samsung.com
6 years, 4 months ago (2014-08-06 18:25:37 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/harpreet.sk@samsung.com/363343005/20001
6 years, 4 months ago (2014-08-06 18:26:03 UTC) #10
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: win_blink_compile_dbg on tryserver.blink ...
6 years, 4 months ago (2014-08-06 20:54:11 UTC) #11
commit-bot: I haz the power
6 years, 4 months ago (2014-08-06 21:19:16 UTC) #12
Message was sent while issue was closed.
Change committed as 179655

Powered by Google App Engine
This is Rietveld 408576698