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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2954433003: [LayoutNG] Make leading OOF objects not to create context

Created:
3 years, 6 months ago by kojii
Modified:
3 years, 6 months ago
Reviewers:
CC:
atotic+reviews_chromium.org, blink-reviews, blink-reviews-layout_chromium.org, cbiesinger, chromium-reviews, dgrogan+ng_chromium.org, eae+blinkwatch, glebl+reviews_chromium.org, jchaffraix+rendering, lchoi+reviews_chromium.org, leviw+renderwatch, ojan+watch_chromium.org, pdr+renderingwatchlist_chromium.org, szager+layoutwatch_chromium.org, zoltan1
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

[LayoutNG] Make leading OOF objects not to create context This reverts the change done in https://codereview.chromium.org/2847823002 BUG=734554 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -37 lines) Patch
M third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp View 2 chunks +3 lines, -25 lines 0 comments Download
M third_party/WebKit/Source/core/layout/ng/inline/ng_inline_layout_algorithm_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/core/layout/ng/ng_block_layout_algorithm_test.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/core/layout/ng/ng_block_node_test.cc View 4 chunks +6 lines, -6 lines 0 comments Download
M third_party/WebKit/Source/core/layout/ng/ng_out_of_flow_layout_part_test.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (2 generated)
kojii
Description was changed from ========== leadoof https://codereview.chromium.org/2847823002 BUG=734554 ========== to ========== leadoof https://codereview.chromium.org/2847823002 BUG=734554 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng ...
3 years, 6 months ago (2017-06-22 05:27:08 UTC) #1
kojii
3 years, 6 months ago (2017-06-22 07:08:53 UTC) #2 
Description was changed from

==========
leadoof

https://codereview.chromium.org/2847823002

BUG=734554
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Make leading OOF objects not to create context

This reverts the change done in https://codereview.chromium.org/2847823002

BUG=734554
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

Powered by Google App Engine
This is Rietveld 408576698