New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 33 users
Status: Started
Owner:
Unavailable Nov 15-30
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature


Sign in to add a comment
[meta] LayoutNG
Project Member Reported by e...@chromium.org, Mar 1 2016 Back to list
Meta bug for implementing LayoutNG: A new layout system designed from the ground up to support the requirements of the web as a platform going forward. The plan is to support both the new and old layout systems in parallel and gradually migrate layout types over to the new system. 

Design doc: https://docs.google.com/document/d/1uxbDh4uONFQOiGuiumlJBLGgO4KDWB8ZEkp7Rd47fw4 
Mailing list: blink_layout_ng@chromium.org 

 
Comment 1 by e...@chromium.org, Mar 16 2016
Current work is on an experimental branch:  https://chromium.googlesource.com/experimental/chromium/src/+/refs/wip/layout/layoutng
Comment 2 by le...@chromium.org, Mar 16 2016
Cc: shans@chromium.org dgro...@chromium.org
Comment 3 by e...@chromium.org, Jun 3 2016
Owner: e...@chromium.org
Summary: [meta] LayoutNG (was: [meta] LayoutNG Experiment)
Comment 4 by e...@chromium.org, Jul 1 2016
Blockedon: 625300
Comment 5 by e...@chromium.org, Jul 6 2016
Description: Show this description
Comment 6 by dk...@chromium.org, Jul 21 2016
Cc: dk...@chromium.org
Project Member Comment 7 by bugdroid1@chromium.org, Aug 1 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d32fdb7a2428764717bddae953628b07eba2afdd

commit d32fdb7a2428764717bddae953628b07eba2afdd
Author: eae <eae@chromium.org>
Date: Mon Aug 01 20:00:32 2016

[LayoutNG] Add NGConstraintSpace class definition

Define initial outline of the constraint space API for LayoutNG by means
of the NG[Derived]ConstraintSpace & NGLayoutOpportunityIterator classes.

BUG=591099
R=ikilpatrick@chromium.org

Review-Url: https://codereview.chromium.org/2176343006
Cr-Commit-Position: refs/heads/master@{#409035}

[modify] https://crrev.com/d32fdb7a2428764717bddae953628b07eba2afdd/third_party/WebKit/Source/core/core.gypi
[add] https://crrev.com/d32fdb7a2428764717bddae953628b07eba2afdd/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.cpp
[add] https://crrev.com/d32fdb7a2428764717bddae953628b07eba2afdd/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h

Project Member Comment 8 by bugdroid1@chromium.org, Aug 1 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8bc705f472c4d11b50b6186a3f057a6b2e45c0f3

commit 8bc705f472c4d11b50b6186a3f057a6b2e45c0f3
Author: ikilpatrick <ikilpatrick@chromium.org>
Date: Mon Aug 01 20:46:10 2016

[LayoutNG] Add blink runtime feature flag for LayoutNG.

BUG=591099

Review-Url: https://codereview.chromium.org/2197843002
Cr-Commit-Position: refs/heads/master@{#409048}

[modify] https://crrev.com/8bc705f472c4d11b50b6186a3f057a6b2e45c0f3/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in

Project Member Comment 9 by bugdroid1@chromium.org, Aug 4 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/98af4a7f6e0cb62818f95c5e37818eb4d447f681

commit 98af4a7f6e0cb62818f95c5e37818eb4d447f681
Author: eae <eae@chromium.org>
Date: Thu Aug 04 17:41:25 2016

[LayoutNG] Change size members on NGConstraintSpace

Update the NGConstraintSpace class to clarify naming and description of
the various size members and methods.

BUG=591099
R=ikilpatrick@chromium.org

Review-Url: https://codereview.chromium.org/2214483002
Cr-Commit-Position: refs/heads/master@{#409824}

[modify] https://crrev.com/98af4a7f6e0cb62818f95c5e37818eb4d447f681/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.cpp
[modify] https://crrev.com/98af4a7f6e0cb62818f95c5e37818eb4d447f681/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h

Blockedon: 635619
Project Member Comment 11 by bugdroid1@chromium.org, Aug 8 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/585b50077748e8a19221d736e307a3cf0c18e0e2

commit 585b50077748e8a19221d736e307a3cf0c18e0e2
Author: ikilpatrick <ikilpatrick@chromium.org>
Date: Mon Aug 08 20:40:32 2016

[LayoutNG] Add NGFragment class definition.

Define initial fragment placeholder. Initially just contains the
inline/block size, and inline/block offset.

BUG=591099

Review-Url: https://codereview.chromium.org/2229583002
Cr-Commit-Position: refs/heads/master@{#410446}

[modify] https://crrev.com/585b50077748e8a19221d736e307a3cf0c18e0e2/third_party/WebKit/Source/core/core.gypi
[add] https://crrev.com/585b50077748e8a19221d736e307a3cf0c18e0e2/third_party/WebKit/Source/core/layout/ng/NGFragment.cpp
[add] https://crrev.com/585b50077748e8a19221d736e307a3cf0c18e0e2/third_party/WebKit/Source/core/layout/ng/NGFragment.h

Project Member Comment 12 by bugdroid1@chromium.org, Aug 8 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/70f373450652eb701351c3a7562ea2c13f737990

commit 70f373450652eb701351c3a7562ea2c13f737990
Author: cbiesinger <cbiesinger@chromium.org>
Date: Mon Aug 08 20:52:13 2016

[layoutng] Rename exclusion types to avoid conflicts with ComputedStyleConstants.h

R=ikilpatrick@chromium.org
BUG=591099

Review-Url: https://codereview.chromium.org/2223163002
Cr-Commit-Position: refs/heads/master@{#410453}

[modify] https://crrev.com/70f373450652eb701351c3a7562ea2c13f737990/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h

Comment 13 by e...@chromium.org, Aug 11 2016
Blockedon: 6122
Comment 14 by e...@chromium.org, Aug 11 2016
Blockedon: 636993
Comment 15 by e...@chromium.org, Aug 25 2016
Description: Show this description
Project Member Comment 16 by bugdroid1@chromium.org, Sep 9 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ac9d3f8a067654f9059e6ab00051be9102ff471e

commit ac9d3f8a067654f9059e6ab00051be9102ff471e
Author: eae <eae@chromium.org>
Date: Fri Sep 09 12:23:04 2016

Initial exclusion aware layout opportunities implementation

Implement exclusion aware layout opportunities algorithm. At the moment
it only handles opportunities for a fixed starting point, sorted by the
width of the opportunities, widest first. Next step is to recompute for
the next exclusion once all current opportunities have been exhausted.

BUG=591099
R=ikilpatrick@chromium.org

Review-Url: https://codereview.chromium.org/2298273002
Cr-Commit-Position: refs/heads/master@{#417562}

[modify] https://crrev.com/ac9d3f8a067654f9059e6ab00051be9102ff471e/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc
[modify] https://crrev.com/ac9d3f8a067654f9059e6ab00051be9102ff471e/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.h
[modify] https://crrev.com/ac9d3f8a067654f9059e6ab00051be9102ff471e/third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc
[modify] https://crrev.com/ac9d3f8a067654f9059e6ab00051be9102ff471e/third_party/WebKit/Source/core/layout/ng/ng_physical_constraint_space.cc
[modify] https://crrev.com/ac9d3f8a067654f9059e6ab00051be9102ff471e/third_party/WebKit/Source/core/layout/ng/ng_physical_constraint_space.h

Project Member Comment 17 by bugdroid1@chromium.org, Sep 9 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1a379b6ecf0b694e1dfe17e9364f3b3ca19bcc0d

commit 1a379b6ecf0b694e1dfe17e9364f3b3ca19bcc0d
Author: eae <eae@chromium.org>
Date: Fri Sep 09 12:58:47 2016

[LayoutNG] Update NGLayoutInputText

Update NGLayoutInputText to include further fields for run segmentation.

BUG=591099
R=drott@chromium.org

Review-Url: https://codereview.chromium.org/2323283002
Cr-Commit-Position: refs/heads/master@{#417567}

[modify] https://crrev.com/1a379b6ecf0b694e1dfe17e9364f3b3ca19bcc0d/third_party/WebKit/Source/core/layout/ng/ng_layout_input_text.h

Blocking: 638156
Comment 19 by e...@chromium.org, Oct 27 2016
Cc: sadrul@chromium.org rbyers@chromium.org mustaq@chromium.org bokan@chromium.org vollick@chromium.org
Issue 417560 has been merged into this issue.
Project Member Comment 20 by bugdroid1@chromium.org, Dec 8 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2c7035b91f520b1d73bce8cb4fe8f72a37560284

commit 2c7035b91f520b1d73bce8cb4fe8f72a37560284
Author: glebl <glebl@chromium.org>
Date: Thu Dec 08 02:01:17 2016

Add W3C CSS 2.1 virtual test suite for LayoutNG.

This patch adds W3C virtual test suite for LayoutNG.

See https://www.w3.org/Style/CSS/Test/Overview.en.html for more information about
Cascading Style Sheets Official W3C Test Suites

BUG=591099

Review-Url: https://codereview.chromium.org/2555963004
Cr-Commit-Position: refs/heads/master@{#437138}

[modify] https://crrev.com/2c7035b91f520b1d73bce8cb4fe8f72a37560284/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/2c7035b91f520b1d73bce8cb4fe8f72a37560284/third_party/WebKit/LayoutTests/VirtualTestSuites
[add] https://crrev.com/2c7035b91f520b1d73bce8cb4fe8f72a37560284/third_party/WebKit/LayoutTests/virtual/layout_ng/css2.1/20110323/README.txt

Comment 21 Deleted
Project Member Comment 22 by bugdroid1@chromium.org, Dec 8 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/17c9aaca8957853d7582f77eed0e29bb5b837679

commit 17c9aaca8957853d7582f77eed0e29bb5b837679
Author: mstensho <mstensho@opera.com>
Date: Thu Dec 08 09:00:17 2016

[LayoutNG] Remove unnecessary #includes

NOTRY=true
BUG=591099

Review-Url: https://codereview.chromium.org/2561553002
Cr-Commit-Position: refs/heads/master@{#437217}

[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/layout_ng_block_flow.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_block_layout_algorithm.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_block_node.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_block_node.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_constraint_space_builder.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_fragment.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_fragment_base.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_fragment_base.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_fragment_builder.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_fragment_builder.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_inline_node.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_layout_coordinator.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_layout_inline_items_builder.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_layout_opportunity_iterator.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_layout_opportunity_tree_node.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_length_utils.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment_base.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment_base.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_physical_text_fragment.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_text_fragment.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_text_layout_algorithm.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_text_layout_algorithm.h
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_units.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_units_test.cc
[modify] https://crrev.com/17c9aaca8957853d7582f77eed0e29bb5b837679/third_party/WebKit/Source/core/layout/ng/ng_writing_mode.cc

Comment 23 by e...@chromium.org, Jan 2 2017
Blockedon: 677839
Comment 24 by surma@chromium.org, Jan 12 2017
Cc: surma@chromium.org
Comment 25 by eco...@igalia.com, Feb 14 2017
Cc: eco...@igalia.com
Comment 26 by e...@chromium.org, Mar 14 2017
Blocking: 700919
Blockedon: 707656
Cc: skobes@chromium.org pdr@chromium.org mmcnulty@chromium.org esprehn@chromium.org chrishtr@chromium.org dglazkov@chromium.org
Issue 422162 has been merged into this issue.
Project Member Comment 29 by bugdroid1@chromium.org, Apr 23 2017
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5d44f961b168a6cff0f822455b03cbca3eb44855

commit 5d44f961b168a6cff0f822455b03cbca3eb44855
Author: kojii <kojii@chromium.org>
Date: Sun Apr 23 23:54:02 2017

[LayoutNG] Add FlagExpectations for LayoutNG 1/3

This patch adds the FlagExpectations file for LayoutNG.

The patch is split into 3 CLs because it hits the max limit of CQ.

There are many flaky tests that may need multiple passes until it is
stabilized.

BUG=591099

Review-Url: https://codereview.chromium.org/2827643004
Cr-Commit-Position: refs/heads/master@{#466571}

[add] https://crrev.com/5d44f961b168a6cff0f822455b03cbca3eb44855/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 30 by bugdroid1@chromium.org, Apr 24 2017
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0a5b389d935f65c3f328ac71dedef9d5699cf49b

commit 0a5b389d935f65c3f328ac71dedef9d5699cf49b
Author: kojii <kojii@chromium.org>
Date: Mon Apr 24 00:50:09 2017

[LayoutNG] Add FlagExpectations for LayoutNG 2/3

This patch adds the FlagExpectations file for LayoutNG.

The patch is split into 3 CLs because it hits the max limit of CQ.

There are many flaky tests that may need multiple passes until it is
stabilized.

BUG=591099

Review-Url: https://codereview.chromium.org/2832393002
Cr-Commit-Position: refs/heads/master@{#466572}

[modify] https://crrev.com/0a5b389d935f65c3f328ac71dedef9d5699cf49b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 31 by bugdroid1@chromium.org, Apr 24 2017
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7808a9b3588b505c51a35aedde04bfe6bc39fe49

commit 7808a9b3588b505c51a35aedde04bfe6bc39fe49
Author: kojii <kojii@chromium.org>
Date: Mon Apr 24 02:11:25 2017

[LayoutNG] Add FlagExpectations for LayoutNG 3/3

This patch adds the FlagExpectations file for LayoutNG.

The patch is split into 3 CLs because it hits the max limit of CQ.

There are many flaky tests that may need multiple passes until it is
stabilized.

BUG=591099

Review-Url: https://codereview.chromium.org/2838483002
Cr-Commit-Position: refs/heads/master@{#466578}

[modify] https://crrev.com/7808a9b3588b505c51a35aedde04bfe6bc39fe49/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 32 by bugdroid1@chromium.org, Apr 25 2017
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/854591636767ea4f9d8076fe0be440cf3a7c0643

commit 854591636767ea4f9d8076fe0be440cf3a7c0643
Author: kojii <kojii@chromium.org>
Date: Tue Apr 25 05:33:00 2017

[LayoutNG] Add flaky results to FlagExpectations for LayoutNG

This patch adds flaky results from 5-6 iterations of running tests.

I will run a few more iterations and see how it stabilizes.

BUG=591099

Review-Url: https://codereview.chromium.org/2836883002
Cr-Commit-Position: refs/heads/master@{#466894}

[modify] https://crrev.com/854591636767ea4f9d8076fe0be440cf3a7c0643/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 33 by bugdroid1@chromium.org, Apr 26 2017
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/61b8dd80224f465e251452cf9ca08bea8cf9d28e

commit 61b8dd80224f465e251452cf9ca08bea8cf9d28e
Author: kojii <kojii@chromium.org>
Date: Wed Apr 26 06:54:36 2017

[LayoutNG] Add 2nd flaky results to FlagExpectations for LayoutNG

This patch adds another flaky results from 5-6 iterations of running
tests.

I will run a few more iterations and see how it stabilizes.

BUG=591099

Review-Url: https://codereview.chromium.org/2840753003
Cr-Commit-Position: refs/heads/master@{#467256}

[modify] https://crrev.com/61b8dd80224f465e251452cf9ca08bea8cf9d28e/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Blocking: 716930
Project Member Comment 35 by bugdroid1@chromium.org, May 1
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/51a5fcdeb62cd950cec7920aa876487afe472082

commit 51a5fcdeb62cd950cec7920aa876487afe472082
Author: kojii <kojii@chromium.org>
Date: Mon May 01 17:30:47 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org

Review-Url: https://codereview.chromium.org/2847493002
Cr-Commit-Position: refs/heads/master@{#468345}

[modify] https://crrev.com/51a5fcdeb62cd950cec7920aa876487afe472082/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 36 by bugdroid1@chromium.org, May 6
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/359ad2233a709573db28dd6255f1f559b02e790f

commit 359ad2233a709573db28dd6255f1f559b02e790f
Author: kojii <kojii@chromium.org>
Date: Sat May 06 18:47:39 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org

Review-Url: https://codereview.chromium.org/2867463002
Cr-Commit-Position: refs/heads/master@{#469845}

[modify] https://crrev.com/359ad2233a709573db28dd6255f1f559b02e790f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 37 by bugdroid1@chromium.org, May 8
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e36ac75e5c539be5271d50805634b98949280660

commit e36ac75e5c539be5271d50805634b98949280660
Author: kojii <kojii@chromium.org>
Date: Mon May 08 06:49:34 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org

Review-Url: https://codereview.chromium.org/2865853002
Cr-Commit-Position: refs/heads/master@{#469909}

[modify] https://crrev.com/e36ac75e5c539be5271d50805634b98949280660/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 38 by bugdroid1@chromium.org, May 12
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e1f4f5e2d1cc9c40e2ab79491606e46d2bc86a90

commit e1f4f5e2d1cc9c40e2ab79491606e46d2bc86a90
Author: kojii <kojii@chromium.org>
Date: Fri May 12 07:57:45 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

Review-Url: https://codereview.chromium.org/2865943002
Cr-Commit-Position: refs/heads/master@{#471245}

[modify] https://crrev.com/e1f4f5e2d1cc9c40e2ab79491606e46d2bc86a90/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 39 by bugdroid1@chromium.org, May 14
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dbbb4e564b1412421f3a7b70cd8fa267581b3dea

commit dbbb4e564b1412421f3a7b70cd8fa267581b3dea
Author: kojii <kojii@chromium.org>
Date: Sun May 14 08:21:10 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Starts removing failures/crashes/timeouts if they pass 20 runs in a row.

This patch removes 1300 lines, meaning 1300 tests were fixed in the last
two weeks.

BUG=591099
TBR=eae@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

Review-Url: https://codereview.chromium.org/2874283005
Cr-Commit-Position: refs/heads/master@{#471619}

[modify] https://crrev.com/dbbb4e564b1412421f3a7b70cd8fa267581b3dea/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 40 by bugdroid1@chromium.org, May 16
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f2db02a021e6a68054c3f8a0fef88ce6584ad9af

commit f2db02a021e6a68054c3f8a0fef88ce6584ad9af
Author: kojii <kojii@chromium.org>
Date: Tue May 16 20:08:53 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2880993003
Cr-Commit-Position: refs/heads/master@{#472198}

[modify] https://crrev.com/f2db02a021e6a68054c3f8a0fef88ce6584ad9af/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 41 by bugdroid1@chromium.org, May 17
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c313139af54d7870871413bf4697f1969569b6d5

commit c313139af54d7870871413bf4697f1969569b6d5
Author: kojii <kojii@chromium.org>
Date: Wed May 17 14:29:58 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2886943004
Cr-Commit-Position: refs/heads/master@{#472448}

[modify] https://crrev.com/c313139af54d7870871413bf4697f1969569b6d5/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 42 by bugdroid1@chromium.org, May 17
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4e3c5b5568c8a4f19d42f8acaa6be76f620f1595

commit 4e3c5b5568c8a4f19d42f8acaa6be76f620f1595
Author: kojii <kojii@chromium.org>
Date: Wed May 17 20:58:45 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2887993004
Cr-Commit-Position: refs/heads/master@{#472562}

[modify] https://crrev.com/4e3c5b5568c8a4f19d42f8acaa6be76f620f1595/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 43 by bugdroid1@chromium.org, May 18
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/92f24bfb4c0523c9b58209666827f12fc80b6c2b

commit 92f24bfb4c0523c9b58209666827f12fc80b6c2b
Author: kojii <kojii@chromium.org>
Date: Thu May 18 14:30:37 2017

[LayoutNG] Update FlagExpectations for LayoutNG

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2886263004
Cr-Commit-Position: refs/heads/master@{#472798}

[modify] https://crrev.com/92f24bfb4c0523c9b58209666827f12fc80b6c2b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 44 by bugdroid1@chromium.org, May 20
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/26e4f40adaad88cf84fb4abb46311fb7a55e1970

commit 26e4f40adaad88cf84fb4abb46311fb7a55e1970
Author: kojii <kojii@chromium.org>
Date: Sat May 20 14:38:25 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Bot results up to 218 are included.

95 tests pass for a week and thus removed.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2900503003
Cr-Commit-Position: refs/heads/master@{#473437}

[modify] https://crrev.com/26e4f40adaad88cf84fb4abb46311fb7a55e1970/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 45 by bugdroid1@chromium.org, May 22
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/38a9bddc6ae85965ef78553fa63e05d35f57f363

commit 38a9bddc6ae85965ef78553fa63e05d35f57f363
Author: kojii <kojii@chromium.org>
Date: Mon May 22 18:30:29 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Bot results from 221 to 228 are included.

New imports for CSS Shapes and CSS Writing Modes adds ~500 failures.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2896703002
Cr-Commit-Position: refs/heads/master@{#473630}

[modify] https://crrev.com/38a9bddc6ae85965ef78553fa63e05d35f57f363/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Blocking: 6606
Cc: -rbyers@chromium.org
Project Member Comment 48 by bugdroid1@chromium.org, May 25
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0a90acb1bae34599f887b69cad29daa243b44b21

commit 0a90acb1bae34599f887b69cad29daa243b44b21
Author: kojii <kojii@chromium.org>
Date: Thu May 25 05:58:44 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Bot results from 230 to 243 are included.

Re-enabled imports for CSS Text adds ~30 failures.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2899963002
Cr-Commit-Position: refs/heads/master@{#474577}

[modify] https://crrev.com/0a90acb1bae34599f887b69cad29daa243b44b21/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 49 by bugdroid1@chromium.org, May 26
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/850d05db1053f7fe28eef34914dbb168b3f022ee

commit 850d05db1053f7fe28eef34914dbb168b3f022ee
Author: kojii <kojii@chromium.org>
Date: Fri May 26 17:46:47 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Bot results from 252 and 254 are included.

400 lines were removed as they pass 37 tries in a row, since May 17.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2897173008
Cr-Commit-Position: refs/heads/master@{#475054}

[modify] https://crrev.com/850d05db1053f7fe28eef34914dbb168b3f022ee/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 50 by bugdroid1@chromium.org, May 29
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fd7049c799296867fa282afcc266239e22ac2be1

commit fd7049c799296867fa282afcc266239e22ac2be1
Author: kojii <kojii@chromium.org>
Date: Mon May 29 14:27:53 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Bot results from 259 to 267 are included.

1180 tests were removed by 25 consecutive passes since 212.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2905383006
Cr-Commit-Position: refs/heads/master@{#475348}

[modify] https://crrev.com/fd7049c799296867fa282afcc266239e22ac2be1/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 51 by bugdroid1@chromium.org, May 30
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/776b9109e21429a28d7cb4b17a581eabcb3ee27f

commit 776b9109e21429a28d7cb4b17a581eabcb3ee27f
Author: kojii <kojii@chromium.org>
Date: Tue May 30 17:31:37 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Bot results from 281 to 289 are included.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2904413003
Cr-Commit-Position: refs/heads/master@{#475581}

[modify] https://crrev.com/776b9109e21429a28d7cb4b17a581eabcb3ee27f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Cc: -esprehn@chromium.org
Project Member Comment 53 by bugdroid1@chromium.org, Jun 2
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/038687b786c7b8b919e07bab90a5b12cfe6e134f

commit 038687b786c7b8b919e07bab90a5b12cfe6e134f
Author: kojii <kojii@chromium.org>
Date: Fri Jun 02 05:48:18 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
289 291 292 294 295 296 297 299 301 302

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2914803002
Cr-Commit-Position: refs/heads/master@{#476588}

[modify] https://crrev.com/038687b786c7b8b919e07bab90a5b12cfe6e134f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 54 by bugdroid1@chromium.org, Jun 4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d149c48bd4cd28bace8e124bae4a57890ecdeb1d

commit d149c48bd4cd28bace8e124bae4a57890ecdeb1d
Author: kojii <kojii@chromium.org>
Date: Sun Jun 04 13:02:39 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
316 317 319 320 329 331

643 tests pass since 252 (May 26) and thus removed.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2922673002
Cr-Commit-Position: refs/heads/master@{#476909}

[modify] https://crrev.com/d149c48bd4cd28bace8e124bae4a57890ecdeb1d/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

For some reason bugdroid hasn't updated this bug with https://codereview.chromium.org/2910883002:

Clean up duplicate tests in web-animations-api

Most of the tests in the LayoutTests/web-animations-api directory have
been upstreamed to the Web Platform Tests and are now redundant with
tests in external/wpt/web-animations. This patch deletes the redundant
test files.

Test files that were added after the push to prepare the tests for
upstreaming, or that were not upstreamed due to a need for further
changes, have been moved to LayoutTests/animations to simplify the test
directory structure.

Some additional references to no-longer-existing tests in the
web-animations-api directory have been removed from FlagExpectations and
SmokeTests. The virtual/stable/web-animations-api tests have not been
touched.

BUG=623437,591099

Review-Url: https://codereview.chromium.org/2910883002
Cr-Commit-Position: refs/heads/master@{#477224}
Committed: https://chromium.googlesource.com/chromium/src/+/c8516e444e4348c99158fa53ea43f73c8a799d29

Project Member Comment 56 by bugdroid1@chromium.org, Jun 7
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c83697f687b19a8e07e9958a3f7a9f03339606de

commit c83697f687b19a8e07e9958a3f7a9f03339606de
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Jun 07 13:16:44 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
334 335 336 338 339 341 342 345 351

27 lines were removed as they pass 24 consecutive runs since 289.

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: I43ff720bf68c8fc094b27a0df092956a0f7a64fb
Reviewed-on: https://chromium-review.googlesource.com/526878
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#477621}
[modify] https://crrev.com/c83697f687b19a8e07e9958a3f7a9f03339606de/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 57 by bugdroid1@chromium.org, Jun 8
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d866518f22fe6fba4b1d9b9ee06388463dbe8d72

commit d866518f22fe6fba4b1d9b9ee06388463dbe8d72
Author: kojii <kojii@chromium.org>
Date: Thu Jun 08 04:55:15 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
359

141 lines were removed as they pass 15 consecutive runs since 316.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2926043004
Cr-Commit-Position: refs/heads/master@{#477894}

[modify] https://crrev.com/d866518f22fe6fba4b1d9b9ee06388463dbe8d72/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 58 by bugdroid1@chromium.org, Jun 8
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/66dd07a24e590b5ea335ce6805f9001ded1c694c

commit 66dd07a24e590b5ea335ce6805f9001ded1c694c
Author: kojii <kojii@chromium.org>
Date: Thu Jun 08 17:22:47 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
367 368 371 373

New 'virtual/exotic-color-space' adds lots of failures.

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2927923002
Cr-Commit-Position: refs/heads/master@{#478008}

[modify] https://crrev.com/66dd07a24e590b5ea335ce6805f9001ded1c694c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 59 by bugdroid1@chromium.org, Jun 9
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f7e09e76687908d294b3a27e41e58448cb88c87e

commit f7e09e76687908d294b3a27e41e58448cb88c87e
Author: Christian Biesinger <cbiesinger@chromium.org>
Date: Fri Jun 09 20:54:56 2017

[layoutng] Implement operator== for constraint spaces

This is a prerequisite for fragment caching, because we
only want to reuse a fragment if the constraint space
hasn't changed.

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ice9bfa8fe2a9cc013997ab92d31e4f1b2df4ea45
Reviewed-on: https://chromium-review.googlesource.com/517203
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#478409}
[modify] https://crrev.com/f7e09e76687908d294b3a27e41e58448cb88c87e/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc
[modify] https://crrev.com/f7e09e76687908d294b3a27e41e58448cb88c87e/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.h
[modify] https://crrev.com/f7e09e76687908d294b3a27e41e58448cb88c87e/third_party/WebKit/Source/core/layout/ng/ng_exclusion.cc
[modify] https://crrev.com/f7e09e76687908d294b3a27e41e58448cb88c87e/third_party/WebKit/Source/core/layout/ng/ng_exclusion.h

Project Member Comment 60 by bugdroid1@chromium.org, Jun 13
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d5f16919458b28796601c76fcb7d2bd0178971ad

commit d5f16919458b28796601c76fcb7d2bd0178971ad
Author: kojii <kojii@chromium.org>
Date: Tue Jun 13 18:57:52 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
378 379 381 384 389 393 397 401

27 lines were removed by 15 consecutive passes since 341 (Jun 6).

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2929393002
Cr-Commit-Position: refs/heads/master@{#479083}

[modify] https://crrev.com/d5f16919458b28796601c76fcb7d2bd0178971ad/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 61 by bugdroid1@chromium.org, Jun 15
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bd7812097cdab94d03a1f38485dfadf5ff72eed1

commit bd7812097cdab94d03a1f38485dfadf5ff72eed1
Author: kojii <kojii@chromium.org>
Date: Thu Jun 15 09:39:48 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
- Local run as of June 15

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2937193002
Cr-Commit-Position: refs/heads/master@{#479649}

[modify] https://crrev.com/bd7812097cdab94d03a1f38485dfadf5ff72eed1/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Is bug 735245 also relevant here?
#62: doesn't look so.
Project Member Comment 64 by bugdroid1@chromium.org, Jun 23
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/20ac216790c8d0a3ba8d6918fd3dff0c591249c4

commit 20ac216790c8d0a3ba8d6918fd3dff0c591249c4
Author: kojii <kojii@chromium.org>
Date: Fri Jun 23 14:20:07 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
510 511 520 523 525 531 543 544

Skipped some results due to instabilities of the bot.

1341 lines removed by 12 consecutive passes since 495 (Jun 20.)

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2951063002
Cr-Commit-Position: refs/heads/master@{#481876}

[modify] https://crrev.com/20ac216790c8d0a3ba8d6918fd3dff0c591249c4/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 65 by bugdroid1@chromium.org, Jun 23
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/55e4d9a48a1965626872aabca38484814f0d063a

commit 55e4d9a48a1965626872aabca38484814f0d063a
Author: kojii <kojii@chromium.org>
Date: Fri Jun 23 20:56:08 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
552

New import added: external/wpt/css/CSS2/text/

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2958613002
Cr-Commit-Position: refs/heads/master@{#482023}

[modify] https://crrev.com/55e4d9a48a1965626872aabca38484814f0d063a/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 66 by bugdroid1@chromium.org, Jun 25
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1ffeff7a49a7aeeb48fc2ac5f3d4f24f6194e85c

commit 1ffeff7a49a7aeeb48fc2ac5f3d4f24f6194e85c
Author: kojii <kojii@chromium.org>
Date: Sun Jun 25 19:40:39 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
564 568 574

99 lines removed by 12 consecutive passes since 510 (Jun 21.)

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2949353006
Cr-Commit-Position: refs/heads/master@{#482175}

[modify] https://crrev.com/1ffeff7a49a7aeeb48fc2ac5f3d4f24f6194e85c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 67 by bugdroid1@chromium.org, Jun 27
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5a0231e8be4e01d38f9c447420c338c57d23b90d

commit 5a0231e8be4e01d38f9c447420c338c57d23b90d
Author: kojii <kojii@chromium.org>
Date: Tue Jun 27 04:41:18 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
576 577 579 584 585 590

153 lines removed by 16 consecutive passes since 529 (Jun 23.)

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2954893003
Cr-Commit-Position: refs/heads/master@{#482538}

[modify] https://crrev.com/5a0231e8be4e01d38f9c447420c338c57d23b90d/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 68 by bugdroid1@chromium.org, Jun 28
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/55911cd8d79455ec434081c1d986b4bad10d0454

commit 55911cd8d79455ec434081c1d986b4bad10d0454
Author: kojii <kojii@chromium.org>
Date: Wed Jun 28 07:40:31 2017

[LayoutNG] Classify test failures

This patch cleanup TestExpectaions by reviewing failures for:
  virtual/layout_ng/external/wpt/css/CSS2/linebox
  virtual/layout_ng/external/wpt/css/CSS2/normal-flow
and make them actionable.

TBR=eae@chromium.org, ikilpatrick@chromium.org
BUG=636993, 591099

Review-Url: https://codereview.chromium.org/2962693003
Cr-Commit-Position: refs/heads/master@{#482914}

[modify] https://crrev.com/55911cd8d79455ec434081c1d986b4bad10d0454/third_party/WebKit/LayoutTests/NeverFixTests
[modify] https://crrev.com/55911cd8d79455ec434081c1d986b4bad10d0454/third_party/WebKit/LayoutTests/TestExpectations

Project Member Comment 69 by bugdroid1@chromium.org, Jun 29
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/033499b1d6f379f9af197faef317b8fa2e43e6c5

commit 033499b1d6f379f9af197faef317b8fa2e43e6c5
Author: kojii <kojii@chromium.org>
Date: Thu Jun 29 04:08:47 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
591 592 593 595 596 597 598 599 603 605 610 615

99 lines removed by 21 consecutive passes since 568 (Jun 25.)

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2960383002
Cr-Commit-Position: refs/heads/master@{#483291}

[modify] https://crrev.com/033499b1d6f379f9af197faef317b8fa2e43e6c5/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Blocking: 700838
Project Member Comment 71 by bugdroid1@chromium.org, Jul 4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e566b8c510fdf37c2ddb2eb9b8de666ce46b544f

commit e566b8c510fdf37c2ddb2eb9b8de666ce46b544f
Author: kojii <kojii@chromium.org>
Date: Tue Jul 04 05:53:43 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
617 619 621 625 626 627 629 632 635 636 639 641

33 lines removed by consecutive passes since 617 (Jun 29.)

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2969953002
Cr-Commit-Position: refs/heads/master@{#484055}

[modify] https://crrev.com/e566b8c510fdf37c2ddb2eb9b8de666ce46b544f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 72 by bugdroid1@chromium.org, Jul 6
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a57ea458887bb6fc364f7865c6de389a3938e31a

commit a57ea458887bb6fc364f7865c6de389a3938e31a
Author: kojii <kojii@chromium.org>
Date: Thu Jul 06 12:51:33 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
643 644 648 649 653 657

33 lines removed by consecutive passes since 639 (Jul 3.)

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2970113003
Cr-Commit-Position: refs/heads/master@{#484559}

[modify] https://crrev.com/a57ea458887bb6fc364f7865c6de389a3938e31a/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 73 by bugdroid1@chromium.org, Jul 7
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/47eb2f424cb82a7b1e4757b0cb48a7e8a73d076a

commit 47eb2f424cb82a7b1e4757b0cb48a7e8a73d076a
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Jul 07 13:56:05 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
664 668

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Change-Id: Ia433dcf298d50022db776956441b1e25048a7f1a
Reviewed-on: https://chromium-review.googlesource.com/563224
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484912}
[modify] https://crrev.com/47eb2f424cb82a7b1e4757b0cb48a7e8a73d076a/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 74 by bugdroid1@chromium.org, Jul 8
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cc07e7e569107ddd6bc833b995c4e7f39691d0f4

commit cc07e7e569107ddd6bc833b995c4e7f39691d0f4
Author: Koji Ishii <kojii@chromium.org>
Date: Sat Jul 08 07:45:13 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
675

The InlineBoxWrapper fix[1] changes the result too much for Rietveld to
handle, and thus moving to Gerrit.

3848 new passes, and 964 crashes turn to failures/timeouts.

[1] https://codereview.chromium.org/2975663002

BUG=591099
TBR=eae@chromium.org
NOTRY=true

Change-Id: I9cce7a01bdacb6bb0f21542344bee1800e7a62ff
Reviewed-on: https://chromium-review.googlesource.com/563768
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485141}
[modify] https://crrev.com/cc07e7e569107ddd6bc833b995c4e7f39691d0f4/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 75 by bugdroid1@chromium.org, Jul 10
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3eb6363c61b669b65ff723f113f5190eb54dc4e4

commit 3eb6363c61b669b65ff723f113f5190eb54dc4e4
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Jul 10 13:54:02 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
681 683 685

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: I16415dc74a3c6a94542a8acae9fa9ca86c0216f2
Reviewed-on: https://chromium-review.googlesource.com/564915
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485257}
[modify] https://crrev.com/3eb6363c61b669b65ff723f113f5190eb54dc4e4/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 76 by bugdroid1@chromium.org, Jul 11
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/72fd03ef3ba18912fad65a49778564392622c2a5

commit 72fd03ef3ba18912fad65a49778564392622c2a5
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Jul 11 13:32:49 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
701 702 703

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: I743f9c1e501ccd3b13e9205060304afbad89b90f
Reviewed-on: https://chromium-review.googlesource.com/566770
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485615}
[modify] https://crrev.com/72fd03ef3ba18912fad65a49778564392622c2a5/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 77 by bugdroid1@chromium.org, Jul 11
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/520552f578754b7e7e7541b38f7747e5bc8c8086

commit 520552f578754b7e7e7541b38f7747e5bc8c8086
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Jul 11 15:58:15 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
704 705

Somehow ng bot is very flaky, hundreds of failures on each run.

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: I68a264c56eb1998354140fe1c6dffde6887d8fe9
Reviewed-on: https://chromium-review.googlesource.com/567019
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485641}
[modify] https://crrev.com/520552f578754b7e7e7541b38f7747e5bc8c8086/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 78 by bugdroid1@chromium.org, Jul 11
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b6f9f400d92872778bb26ebe2a9222d02866a61c

commit b6f9f400d92872778bb26ebe2a9222d02866a61c
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Jul 11 16:56:35 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
711

300 more failures after the last update. Probably tests exiting early.
I'll try local run next.

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: Ice558b48a750940e9f6e9600df14f63bf1a4e6ae
Reviewed-on: https://chromium-review.googlesource.com/566986
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485654}
[modify] https://crrev.com/b6f9f400d92872778bb26ebe2a9222d02866a61c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Blocking: 507397
Project Member Comment 80 by bugdroid1@chromium.org, Jul 12
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f62c1e728414503598d610e0b030dfd6ebb45c8b

commit f62c1e728414503598d610e0b030dfd6ebb45c8b
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Jul 12 11:34:23 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
723 724 728

884 lines were removed by consecutive passes since 643 (Jul 6),
excluding the bot troubles yesterday (701-711). This should undo all
additions during the bot troubles.

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: I5c62411bf0a3b32a5c1c95fc94c2d0c5dd3a6d10
Reviewed-on: https://chromium-review.googlesource.com/567939
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485927}
[modify] https://crrev.com/f62c1e728414503598d610e0b030dfd6ebb45c8b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 81 by bugdroid1@chromium.org, Jul 16
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5c0480b40f22b4f68da2160c0ea343cdb6048ae9

commit 5c0480b40f22b4f68da2160c0ea343cdb6048ae9
Author: Koji Ishii <kojii@chromium.org>
Date: Sun Jul 16 10:58:58 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
730 739 740 741 742 745 746

4022 lines were removed by consecutive passes since 681 (Jul 10).

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Bug: 
Change-Id: I22cabe0f1175ddab6449e18d70ef63cc2e09d537
Reviewed-on: https://chromium-review.googlesource.com/572485
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487009}
[modify] https://crrev.com/5c0480b40f22b4f68da2160c0ea343cdb6048ae9/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Cc: -sadrul@chromium.org
Project Member Comment 83 by bugdroid1@chromium.org, Jul 18
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ca1fc4282ee7bd4274078af37c3145f19e290de4

commit ca1fc4282ee7bd4274078af37c3145f19e290de4
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Jul 18 19:18:22 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
753 754 755 757 760 761 762 763 765 766 767

41 lines were removed by consecutive passes since 730.

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: Ic324594a8de627cd43e42c4835e9e1283165fabe
Reviewed-on: https://chromium-review.googlesource.com/575314
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487551}
[modify] https://crrev.com/ca1fc4282ee7bd4274078af37c3145f19e290de4/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 84 by bugdroid1@chromium.org, Jul 21
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c2ed001d0b093fe7feb35c1c8685ea5be5451784

commit c2ed001d0b093fe7feb35c1c8685ea5be5451784
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Jul 21 07:00:56 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
769 775 781 782 785 788 789 790 791 792 795 796 798 799

155 lines were removed by consecutive passes since 753.

TBR=eae@chromium.org
NOTRY=true

BUG: 591099
Change-Id: Ib343eeaf4540ed94747f68724adedcecc175dac4
Reviewed-on: https://chromium-review.googlesource.com/580732
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488608}
[modify] https://crrev.com/c2ed001d0b093fe7feb35c1c8685ea5be5451784/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 85 by bugdroid1@chromium.org, Jul 26
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8355b6fc36393a09a80eea00f9573f0be79b0b06

commit 8355b6fc36393a09a80eea00f9573f0be79b0b06
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Jul 26 09:43:35 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
801 804 80{6..9} 812 813 815 82{2..3} 827 829 833 836 838 841 84{3..9}

419 lines were removed by consecutive passes since 806.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I9b439f566de512a684542056dbc47f90629f5310
Reviewed-on: https://chromium-review.googlesource.com/584697
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489585}
[modify] https://crrev.com/8355b6fc36393a09a80eea00f9573f0be79b0b06/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 86 by bugdroid1@chromium.org, Jul 28
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7f8fd3222bac91dcca7bb73a6779ff95a71298fc

commit 7f8fd3222bac91dcca7bb73a6779ff95a71298fc
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Jul 28 14:35:08 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
85{1..4} 85{6..7} 86{0..3} 86{5..7} 869 873 875

3 lines were removed by consecutive passes since 841.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I607742a8ca3fce9b412111dc44b6705f6c422c51
Reviewed-on: https://chromium-review.googlesource.com/590636
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490386}
[modify] https://crrev.com/7f8fd3222bac91dcca7bb73a6779ff95a71298fc/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 87 by bugdroid1@chromium.org, Jul 28
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/133f781660563dac94b5cce3f17bf61b45c82526

commit 133f781660563dac94b5cce3f17bf61b45c82526
Author: Takeshi Yoshino <tyoshino@chromium.org>
Date: Fri Jul 28 18:05:07 2017

Remove a non-existent layout test from FlagExpectations/enable-blink-features=LayoutNG

fast/text/atomic-inline-before-ellipsis.html was added in
https://chromium.googlesource.com/chromium/src/+/7f8fd3222bac91dcca7bb73a6779ff95a71298fc%5E%21/
but it doesn't exist.

I just got a presubmit error for this (not sure why as I didn't
touch anything around this).

It was added by https://chromium-review.googlesource.com/c/579973/
but the patch has been reverted.

Bug: 591099, 745867
Change-Id: Idc4eb3530d49cac399bcbac8457f3bab58d59840

TBR=kojii@chromium.org

Change-Id: Idc4eb3530d49cac399bcbac8457f3bab58d59840
Reviewed-on: https://chromium-review.googlesource.com/592087
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490461}

Project Member Comment 88 by bugdroid1@chromium.org, Aug 3
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b4ca9242fe901e87e9d732a36fc789716c2e2bb7

commit b4ca9242fe901e87e9d732a36fc789716c2e2bb7
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Aug 03 14:39:36 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
88{0..2} 884 888 896

68 lines were removed by consecutive passes since 873.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ic80527887a39c4e73fbf351fc22f01bef4fc54aa
Reviewed-on: https://chromium-review.googlesource.com/600447
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491735}
[modify] https://crrev.com/b4ca9242fe901e87e9d732a36fc789716c2e2bb7/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 89 by bugdroid1@chromium.org, Aug 7
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/831ded9bc26cbf99805cac66893d8c24eec22388

commit 831ded9bc26cbf99805cac66893d8c24eec22388
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Aug 07 15:51:10 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
898 902 91{1..2} 91{6..7} 92{8..9}

31 lines were removed by consecutive passes since 896.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Iacc22e37cd743f88d1ba72a5b25ec2d22b0e0632
Reviewed-on: https://chromium-review.googlesource.com/603927
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492329}
[modify] https://crrev.com/831ded9bc26cbf99805cac66893d8c24eec22388/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 90 by bugdroid1@chromium.org, Aug 14
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eb256331b54b5023e312a2ea563c3f5d777069f5

commit eb256331b54b5023e312a2ea563c3f5d777069f5
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Aug 14 11:40:07 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
93{0..8} 94{1..2} 94{6..7} 950 95{2..5} 95{7..8} 961 96{3..6}

513 lines were removed by consecutive passes since 952.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Icf59072d18a8742e45fb0f5d1c81a2e5e243552f
Reviewed-on: https://chromium-review.googlesource.com/612934
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494034}
[modify] https://crrev.com/eb256331b54b5023e312a2ea563c3f5d777069f5/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 91 by bugdroid1@chromium.org, Aug 22
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/73fb33b077df8bb44d7d7270cde46f448c0a47e7

commit 73fb33b077df8bb44d7d7270cde46f448c0a47e7
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Aug 22 01:56:09 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1015

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I448521951e4631dc26fc6ef25f87b6fc8618fb4b
Reviewed-on: https://chromium-review.googlesource.com/624820
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496171}
[modify] https://crrev.com/73fb33b077df8bb44d7d7270cde46f448c0a47e7/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 92 by bugdroid1@chromium.org, Aug 25
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3ac51fd1e6d35581c8b36c0753beb2358be49b3c

commit 3ac51fd1e6d35581c8b36c0753beb2358be49b3c
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Aug 25 07:11:23 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1092 1094

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I7013c86031700c6cedb0b3091df9c40c02467844
Reviewed-on: https://chromium-review.googlesource.com/634765
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#497342}
[modify] https://crrev.com/3ac51fd1e6d35581c8b36c0753beb2358be49b3c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 93 by bugdroid1@chromium.org, Aug 28
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c410e6a20f161e8e312260f0b3beb0c26dfc5cb6

commit c410e6a20f161e8e312260f0b3beb0c26dfc5cb6
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Aug 28 18:19:05 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
109{5..6} 109{8..9} 1100 110{5..6}

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I45557090313ad8131b8f7aae4b5d32f436762659
Reviewed-on: https://chromium-review.googlesource.com/638111
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#497807}
[modify] https://crrev.com/c410e6a20f161e8e312260f0b3beb0c26dfc5cb6/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 94 by bugdroid1@chromium.org, Sep 1
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ac8f154d975080458b32dc0f6642f8f0120ceabd

commit ac8f154d975080458b32dc0f6642f8f0120ceabd
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Sep 01 01:37:03 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
110{7..9} 1110 1112 111{5..8} 112{3..5} 112{7..8} 113{0..1} 1134

131 lines were removed by consecutive passes since 1107.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ic68654aaf80ae7fe598d5fb73cf895c1beab62b4
Reviewed-on: https://chromium-review.googlesource.com/647046
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499104}
[modify] https://crrev.com/ac8f154d975080458b32dc0f6642f8f0120ceabd/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 95 by bugdroid1@chromium.org, Sep 4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ffe7ca0ca25adad3b9ff455551a70dd98b60067b

commit ffe7ca0ca25adad3b9ff455551a70dd98b60067b
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Sep 04 04:59:31 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1123 1124 1125 1127 1128 1130 1131 1134 1138 1140 1142 1143 1144 1147
1148 1149 1151 1152 1154 1158 1159 1161 1162 1163

12 lines were removed by consecutive passes since 1138.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ic986729e41d75d74adcc8b48a1d490f85b6c4394
Reviewed-on: https://chromium-review.googlesource.com/647412
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499437}
[modify] https://crrev.com/ffe7ca0ca25adad3b9ff455551a70dd98b60067b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Blockedon: 758816
Project Member Comment 97 by bugdroid1@chromium.org, Sep 7
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5a30852b8847ee67d852c1a411ac9461154c67b7

commit 5a30852b8847ee67d852c1a411ac9461154c67b7
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Sep 07 19:51:17 2017

[LayoutNG] Add NGPhysicalContainerFragment

This patch adds NGPhysicalContainerFragment, a fragment that has
children, to share code between NGPhysicalBoxFragment and
NGPhysicalLineBoxFragment.

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ic72db2cf818f24e79402138caa8e885607dca591
Reviewed-on: https://chromium-review.googlesource.com/655018
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500359}
[modify] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/BUILD.gn
[modify] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.cc
[modify] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_line_box_fragment.h
[modify] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/ng_physical_box_fragment.cc
[modify] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/ng_physical_box_fragment.h
[add] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/ng_physical_container_fragment.cc
[add] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/ng_physical_container_fragment.h
[modify] https://crrev.com/5a30852b8847ee67d852c1a411ac9461154c67b7/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment.h

Project Member Comment 98 by bugdroid1@chromium.org, Sep 8
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/186ce218a50e4122436c3679412b35c4dd48c95d

commit 186ce218a50e4122436c3679412b35c4dd48c95d
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Sep 08 04:08:57 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1169 1170 1171 1172 1173 1177 1178 1181 1182 1183 1186 1188 1189 1191
1192 1195 1196 1199 1200 1201 1202 1203 1206 1213 1214

2 lines were removed by consecutive passes since 1165.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ib6ff939974956ca02189c8df71b07daf0b44d4b7
Reviewed-on: https://chromium-review.googlesource.com/656642
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500502}
[modify] https://crrev.com/186ce218a50e4122436c3679412b35c4dd48c95d/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 99 by bugdroid1@chromium.org, Sep 10
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7d5a4b357b68d530db515f2488f54569c2c3232f

commit 7d5a4b357b68d530db515f2488f54569c2c3232f
Author: Koji Ishii <kojii@chromium.org>
Date: Sun Sep 10 16:53:59 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1181 1182 1183 1186 1188 1189 1191 1192 1195 1196 1199 1200 1201 1202
1203 1206 1213 1214 1217 1219 1220 1221 1222 1223 1227 1228 1229 1238
1239 1240 1241 1246 1247

26 lines were removed by consecutive passes since 1217.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ib43ec8d1a3e9dbd05e3e4c5a9f15b1f578d2fc61
Reviewed-on: https://chromium-review.googlesource.com/658678
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500813}
[modify] https://crrev.com/7d5a4b357b68d530db515f2488f54569c2c3232f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 100 by bugdroid1@chromium.org, Sep 12
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e96e7e2e8bb365e4a9947a1b44c3fafb9577905c

commit e96e7e2e8bb365e4a9947a1b44c3fafb9577905c
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Sep 12 15:35:31 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1259 1260 1261 1262 1263 1264 1266 1268 1269 1270 1271 1272 1273
1275 1277 1278

Skipped results before mstensho's adjustments.

75 lines were removed by consecutive passes since 1229.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I3ce8ef09ca8ddda3b2c04e59c679eb93728717c4
Reviewed-on: https://chromium-review.googlesource.com/663497
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501286}
[modify] https://crrev.com/e96e7e2e8bb365e4a9947a1b44c3fafb9577905c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 101 by bugdroid1@chromium.org, Sep 13
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a43ad2189e67d66783cfa5427e10a15ee8cacf45

commit a43ad2189e67d66783cfa5427e10a15ee8cacf45
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Sep 13 15:26:31 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1281 1282 1283 1284 1285 1286 1287 1291 1293 1294 1299 1300 1302
1303 1304 1305

85 lines were removed by consecutive passes since 1248.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I479d8f2513097eea14cd12eb8e4188b14b1c8bde
Reviewed-on: https://chromium-review.googlesource.com/665197
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501637}
[modify] https://crrev.com/a43ad2189e67d66783cfa5427e10a15ee8cacf45/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 102 by bugdroid1@chromium.org, Sep 14
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/25610144a23efc46424e7dc97730926ca3fc9d70

commit 25610144a23efc46424e7dc97730926ca3fc9d70
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Sep 14 15:22:38 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1308 1310 1312 1313 1317 1318 1319 1320 1322 1323 1324 1325

7329 lines were removed by consecutive passes since 1305.

From this CL, the removal period was shorten to 24 hours.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Iad205e8dd951ead750a76d1b94cd4550ad64d504
Reviewed-on: https://chromium-review.googlesource.com/667297
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501954}
[modify] https://crrev.com/25610144a23efc46424e7dc97730926ca3fc9d70/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 103 by bugdroid1@chromium.org, Sep 15
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/57031d8418af9de8520ea3af9403b460f16e731c

commit 57031d8418af9de8520ea3af9403b460f16e731c
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Sep 15 14:11:15 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1330 1335 1337 1338 1339

63 lines were removed by consecutive passes since 1325.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I4f7d89736c336558a53bf57a37297eb1f29df378
Reviewed-on: https://chromium-review.googlesource.com/668456
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502244}
[modify] https://crrev.com/57031d8418af9de8520ea3af9403b460f16e731c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 104 by bugdroid1@chromium.org, Sep 19
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/142cc4ec878787f9cdd3ca5d9a1c4fc949278520

commit 142cc4ec878787f9cdd3ca5d9a1c4fc949278520
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Sep 19 06:09:34 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1340 1341 1343 1346 1347 1352 1353

504 lines were removed by consecutive passes since 1340.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ie7f4402d4dede92ada0ff90be632d1828b845cc4
Reviewed-on: https://chromium-review.googlesource.com/670884
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502787}
[modify] https://crrev.com/142cc4ec878787f9cdd3ca5d9a1c4fc949278520/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 105 by bugdroid1@chromium.org, Sep 21
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3dfc97fb331940041454ead54ea6b6b10f5500b1

commit 3dfc97fb331940041454ead54ea6b6b10f5500b1
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Sep 21 13:30:45 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1355 1356 1357 1359 1360 1361

11 lines were removed by consecutive passes since 1355.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I97c1b12aad8acac71f384adcea10cb4833e6db97
Reviewed-on: https://chromium-review.googlesource.com/676825
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#503431}
[modify] https://crrev.com/3dfc97fb331940041454ead54ea6b6b10f5500b1/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 106 by bugdroid1@chromium.org, Sep 25
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8b7d15de48cda888891b676534661ec215b25f94

commit 8b7d15de48cda888891b676534661ec215b25f94
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Sep 25 13:50:51 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1362 1363 1370 1371 1373 1378 1381

238 lines were removed by consecutive passes since 1370.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I583ee30a4c19755dd04f18412a2a33a96ecaf2a6
Reviewed-on: https://chromium-review.googlesource.com/681736
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504042}
[modify] https://crrev.com/8b7d15de48cda888891b676534661ec215b25f94/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 107 by bugdroid1@chromium.org, Sep 26
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7e604982ddd8d0c4004ed618c90080be42f13a14

commit 7e604982ddd8d0c4004ed618c90080be42f13a14
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Sep 26 11:27:12 2017

Split "ordinal value" logic from LayoutListItem

This patch splits the "ordinal value" logic[1] to a separate class,
ListItemOridinal, so that we can re-use the logic in LayoutNG.

The logic is mostly DOM operations that moving them out of layout is
reasonable from the layering perspective too.

It still relies on layout tree to provide the storage, and events for
insertions and deletions, since any elements with 'display: list-item'
can be list items.

[1] https://html.spec.whatwg.org/multipage/grouping-content.html#ordinal-value

Bug: 591099
Change-Id: Ia61ee764db83203ac8a4006cbab534b5ccf56606
Reviewed-on: https://chromium-review.googlesource.com/678415
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504338}
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/html/BUILD.gn
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/html/HTMLLIElement.cpp
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/html/HTMLLIElement.h
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/html/HTMLOListElement.cpp
[add] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/html/ListItemOrdinal.cpp
[add] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/html/ListItemOrdinal.h
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/layout/LayoutCounter.cpp
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/layout/LayoutListItem.cpp
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/layout/LayoutListItem.h
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/layout/LayoutListMarker.cpp
[modify] https://crrev.com/7e604982ddd8d0c4004ed618c90080be42f13a14/third_party/WebKit/Source/core/layout/api/LayoutLIItem.h

Project Member Comment 108 by bugdroid1@chromium.org, Sep 28
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e544c4244f4c6a9328c7d0fd36892648df288f9b

commit e544c4244f4c6a9328c7d0fd36892648df288f9b
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Sep 28 06:44:06 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1385 1386 1387 1388 1389 1396 1398 1409 1410 1415 1419 1423 1433 1434

6 lines were removed by consecutive passes since 1410.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: If4d5d190d62ffab834d003ad71c7e82e9a4a5182
Reviewed-on: https://chromium-review.googlesource.com/688260
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504926}
[modify] https://crrev.com/e544c4244f4c6a9328c7d0fd36892648df288f9b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 109 by bugdroid1@chromium.org, Oct 2
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8cb8256df002f0e0ebaea387315bc4e00491d6b3

commit 8cb8256df002f0e0ebaea387315bc4e00491d6b3
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Oct 02 08:22:38 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1441 1442 1443 1444 1445 1446 1447 1448 1450 1454 1459

36 lines were removed by consecutive passes since 1443.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I58883b7d7237885f2888949c95021cfe340af59a
Reviewed-on: https://chromium-review.googlesource.com/695041
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505558}
[modify] https://crrev.com/8cb8256df002f0e0ebaea387315bc4e00491d6b3/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 110 by bugdroid1@chromium.org, Oct 4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5416e45d5421ebed76992fee8d9baec7e08b7ca9

commit 5416e45d5421ebed76992fee8d9baec7e08b7ca9
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Oct 04 04:54:33 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1460 1464 1466 1471 1474 1474 1476 1477 1484 1489

13 lines were removed by consecutive passes since 1471.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Iaf1f96a7ff3fef155caf3e651a5bb52ece6a646c
Reviewed-on: https://chromium-review.googlesource.com/700134
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506303}
[modify] https://crrev.com/5416e45d5421ebed76992fee8d9baec7e08b7ca9/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 111 by bugdroid1@chromium.org, Oct 6
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eff2364e27c71e9becf6c634842dc271c10bfe6e

commit eff2364e27c71e9becf6c634842dc271c10bfe6e
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Oct 06 20:09:43 2017

[LayoutNG] Add NGLogicalRect

This patch adds NGLogicalRect to compute visual rect in logical
coordinates.

In doing this, minor adjustments are done to NGLogicalOffset:
1. Add operator+/- for NGLogicalSize.
2. Whether "offset - offset" should produce "size" or "offset" was
   controversial. Added two functions to clarify, "RelativeTo" and
   "DistanceTo", and keep the "operator-" unchanged as the alias to
   "offset" version.

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ib3e5450cdf040dcffc50b6e3f69cc00d52338780
Reviewed-on: https://chromium-review.googlesource.com/701817
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@opera.com>
Cr-Commit-Position: refs/heads/master@{#507157}
[modify] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/BUILD.gn
[modify] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/layout/BUILD.gn
[modify] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc
[modify] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.h
[add] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_rect.cc
[add] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_rect.h
[add] https://crrev.com/eff2364e27c71e9becf6c634842dc271c10bfe6e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_rect_test.cc

Project Member Comment 112 by bugdroid1@chromium.org, Oct 9
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/668d56b9465445c01c1c374174c624f02464887e

commit 668d56b9465445c01c1c374174c624f02464887e
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Oct 09 19:36:27 2017

[LayoutNG] NGLogicalDelta resolves the ambiguity of subtractions

Following the ambiguity discussion in r507157 (CL:701817):

"offset - offset" is ambiguous because both of below are true:
  offset + offset = offset
  offset + size = offset

NGLogicalDelta resolves this ambiguity by allowing implicit
conversions both to NGLogicalOffset and to NGLogicalSize.

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I725ba1cfdfc1a78a7754f01a96919589ceffb0bf
Reviewed-on: https://chromium-review.googlesource.com/706834
Commit-Queue: Koji Ishii <kojii@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@opera.com>
Cr-Commit-Position: refs/heads/master@{#507449}
[modify] https://crrev.com/668d56b9465445c01c1c374174c624f02464887e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc
[modify] https://crrev.com/668d56b9465445c01c1c374174c624f02464887e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.h
[modify] https://crrev.com/668d56b9465445c01c1c374174c624f02464887e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_rect.cc
[modify] https://crrev.com/668d56b9465445c01c1c374174c624f02464887e/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_size.h

Project Member Comment 113 by bugdroid1@chromium.org, Oct 10
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ceb9d3e891f1b95d90aa6e9976d4b33049f50326

commit ceb9d3e891f1b95d90aa6e9976d4b33049f50326
Author: Xiaocheng Hu <xiaochengh@chromium.org>
Date: Tue Oct 10 02:15:57 2017

Add LayoutObject::EnclosingNGBlockFlow

This patch adds a shorthand LayoutObject::EnclosingNGBlockFlow to
detect if a LayoutObject is laid-out with LayoutNG, and if so, returns
the enclosing box as a LayoutNGBlockFlow.

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ib03c15a6dfa786bb16f52c45ca0d491b2c269d86
Reviewed-on: https://chromium-review.googlesource.com/707766
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#507572}
[modify] https://crrev.com/ceb9d3e891f1b95d90aa6e9976d4b33049f50326/third_party/WebKit/Source/core/layout/LayoutObject.cpp
[modify] https://crrev.com/ceb9d3e891f1b95d90aa6e9976d4b33049f50326/third_party/WebKit/Source/core/layout/LayoutObject.h
[modify] https://crrev.com/ceb9d3e891f1b95d90aa6e9976d4b33049f50326/third_party/WebKit/Source/core/layout/ng/inline/ng_inline_node.cc

Project Member Comment 114 by bugdroid1@chromium.org, Oct 10
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bee84c3d0952eb52fbc0e537f8c7dbddd275001d

commit bee84c3d0952eb52fbc0e537f8c7dbddd275001d
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Oct 10 05:01:40 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1495 1498 1500 1502 1506 1508 1509 1511 1512 1513 1514 1516 1517 1519
1523 1524 1525 1526 1527 1529 1530 1533 1534 1535 1536 1537 1538 1539
1540 1541 1542 1543 1545 1546 1547 1551 1552 1553 1556 1557 1558

69 lines were removed by consecutive passes since 1546.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I0786902ebd305bfcc3c6ed923b7d03d8129ef236
Reviewed-on: https://chromium-review.googlesource.com/708474
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#507590}
[modify] https://crrev.com/bee84c3d0952eb52fbc0e537f8c7dbddd275001d/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 115 by bugdroid1@chromium.org, Oct 11
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aeecbdf360107fb5186c12ebc33e1f570f7f934b

commit aeecbdf360107fb5186c12ebc33e1f570f7f934b
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Oct 11 09:15:23 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1561 1562 1563 1564 1566 1568 1570 1571 1572 1575 1576 1577 1578 1579
1580

21 lines were removed by consecutive passes since 1559.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I6d0169a738cd56f19de09c7932494438b412cc76
Reviewed-on: https://chromium-review.googlesource.com/711739
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#507933}
[modify] https://crrev.com/aeecbdf360107fb5186c12ebc33e1f570f7f934b/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 116 by bugdroid1@chromium.org, Oct 12
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/62d8fd162a7c209c193090154eeba50d664fa02f

commit 62d8fd162a7c209c193090154eeba50d664fa02f
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Oct 12 05:43:15 2017

[LayoutNG] Cleanup visual overflow computations

This patch cleans up visual overflow computations in:
1. Use type-safe geometry classes. NGPhysicalOffsetRect is added.
2. Add conversion functions for LayoutUnit to type-safe geometry
   classes.
3. LocalVisualRect() turned out not to include its children.
   Matched to the definition and separated ContentsVisualRect().
4. Removed NGPhysicalFragment::LocalVisualRect() and its storage.
   NGPhysicalBoxFragment has the storage, but other types are
   assumed to be cheap and compute on the fly.
5. Removed NGPhysicalFragment::UpdateVisualRect(). Box fragments
   computes in NGFragmentBuilder.
6. With that, removed all virtual functions from NGPhysicalFragment.

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Iaa39ad1260e5bd34b430bb3693d914a9e73f8e46
Reviewed-on: https://chromium-review.googlesource.com/691476
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508261}
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/BUILD.gn
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/BUILD.gn
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_offset.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_offset.h
[add] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_offset_rect.cc
[add] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_offset_rect.h
[add] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_offset_rect_test.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_size.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/geometry/ng_physical_size.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/inline/ng_inline_layout_algorithm_test.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/inline/ng_line_box_fragment_builder.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_text_fragment.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/inline/ng_physical_text_fragment.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/inline/ng_text_fragment_builder.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/layout_ng_block_flow.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/layout_ng_block_flow.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_fragment_builder.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_fragment_builder.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_physical_box_fragment.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_physical_box_fragment.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_physical_container_fragment.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_physical_container_fragment.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment.cc
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment.h
[modify] https://crrev.com/62d8fd162a7c209c193090154eeba50d664fa02f/third_party/WebKit/Source/core/paint/ng/ng_paint_fragment.cc

Project Member Comment 117 by bugdroid1@chromium.org, Oct 12
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3382647c66210bcf3c0024c2763f3d838932a96f

commit 3382647c66210bcf3c0024c2763f3d838932a96f
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Oct 12 16:05:07 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1589 1591 1595 1596 1597 1598 1600 1605 1606 1607 1609 1610 1611 1615
1616 1617 1618 1619 1620

4 lines were removed by consecutive passes since 1589.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I1d642667fb433f63a20204422252ab3bb5806325
Reviewed-on: https://chromium-review.googlesource.com/716436
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508361}
[modify] https://crrev.com/3382647c66210bcf3c0024c2763f3d838932a96f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 118 by bugdroid1@chromium.org, Oct 16
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6d39dc71dd2cc21af084f491cf4c1e1128dd84d8

commit 6d39dc71dd2cc21af084f491cf4c1e1128dd84d8
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Oct 16 02:32:30 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1662 1665 1668 1669 1670 1671

This is a large change after DCHECK was enabled (crbug.com/774299).

TBR=eae@chromium.org
NOTRY=true

Bug: 591099, 774299
Change-Id: I219b6ffd787dfd62a3985e2a469f23eba9f4abb0
Reviewed-on: https://chromium-review.googlesource.com/720416
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508971}
[modify] https://crrev.com/6d39dc71dd2cc21af084f491cf4c1e1128dd84d8/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 119 by bugdroid1@chromium.org, Oct 18
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5c161d39d0623149c4a51837973f2136ea9d2114

commit 5c161d39d0623149c4a51837973f2136ea9d2114
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Oct 18 04:59:38 2017

[LayoutNG] Add BoxType to distinguish different box fragments

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I2f6d01a6351b07d8bc93ad016858f49f549cf89c
Reviewed-on: https://chromium-review.googlesource.com/722383
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509687}
[modify] https://crrev.com/5c161d39d0623149c4a51837973f2136ea9d2114/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment.cc
[modify] https://crrev.com/5c161d39d0623149c4a51837973f2136ea9d2114/third_party/WebKit/Source/core/layout/ng/ng_physical_fragment.h

Project Member Comment 120 by bugdroid1@chromium.org, Oct 18
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/11fa3902e0ebaeca45ce76b148add69b3f43a5ce

commit 11fa3902e0ebaeca45ce76b148add69b3f43a5ce
Author: Koji Ishii <kojii@chromium.org>
Date: Wed Oct 18 09:42:47 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1673 1674 1676 1677 1678 1680 1681 1685 1687 1688 1691 1696 1697
1699 1700 1701 1702 1709 1712 1720 1722 1723 1725 1731

30 lines were removed by consecutive passes since 1685.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ib64f101f2ef3ce6c10d56fab65fe6b352f76fedc
Reviewed-on: https://chromium-review.googlesource.com/720859
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509725}
[modify] https://crrev.com/11fa3902e0ebaeca45ce76b148add69b3f43a5ce/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 121 by bugdroid1@chromium.org, Oct 19
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/71de49546d04b2d6a023c37e9cb3b6551aebb638

commit 71de49546d04b2d6a023c37e9cb3b6551aebb638
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Oct 19 05:05:53 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1734 1735 1737 1742 1747 1750 1754 1758 1759 1762 1765 1768 1769 1771

0 lines were removed by consecutive passes since 1720.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I2ab352f0846b5ae31b7834ab9634326580fe3488
Reviewed-on: https://chromium-review.googlesource.com/727486
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510001}
[modify] https://crrev.com/71de49546d04b2d6a023c37e9cb3b6551aebb638/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 122 by bugdroid1@chromium.org, Oct 20
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/492e5385d984d3df932c97df2511f1bd6b702f33

commit 492e5385d984d3df932c97df2511f1bd6b702f33
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Oct 20 16:04:07 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1774 1775 1776 1779 1782 1784 1789 1791 1792 1794 1795 1805 1806
1809 1812 1814

50 lines were removed by consecutive passes since 1784.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Iec2731830b4148f32de0e37fde9b16ae3932f2b9
Reviewed-on: https://chromium-review.googlesource.com/730904
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510455}
[modify] https://crrev.com/492e5385d984d3df932c97df2511f1bd6b702f33/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 123 by bugdroid1@chromium.org, Oct 23
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d8b8ac9c772143e15da70972ac62d4174eecf290

commit d8b8ac9c772143e15da70972ac62d4174eecf290
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Oct 23 09:38:04 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1816 1817 1818 1819 1820 1822 1823 1825 1826 1827 1829 1831 1832 1833
1842

28 lines were removed by consecutive passes since 1814.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I80ffdc3acd8ecb5f459f56400e640cf88ba56f6f
Reviewed-on: https://chromium-review.googlesource.com/732857
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510748}
[modify] https://crrev.com/d8b8ac9c772143e15da70972ac62d4174eecf290/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 124 by bugdroid1@chromium.org, Oct 24
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b6233fb86d4f2ff3c13d69458373e711c6afa95d

commit b6233fb86d4f2ff3c13d69458373e711c6afa95d
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Oct 24 09:45:05 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1845 1846 1851 1852 1855 1856 1857 1858 1859 1863

91 lines were removed by consecutive passes since 1842.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I4a4cdaaac959edcf2840e21c2690148f48d20198
Reviewed-on: https://chromium-review.googlesource.com/735161
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#511070}
[modify] https://crrev.com/b6233fb86d4f2ff3c13d69458373e711c6afa95d/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 125 by bugdroid1@chromium.org, Oct 26
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e896f891c4c31fd21b36cf388b1786ec141ed817

commit e896f891c4c31fd21b36cf388b1786ec141ed817
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Oct 26 07:01:34 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1871 1874 1875 1876 1880 1885 1889 1890 1891 1892 1893 1895 1896 1897
1898 1899

6 lines were removed by consecutive passes since 1871.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ib2678cd0b6a37530829f15e1eb4306e40597721c
Reviewed-on: https://chromium-review.googlesource.com/737501
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#511754}
[modify] https://crrev.com/e896f891c4c31fd21b36cf388b1786ec141ed817/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 126 by bugdroid1@chromium.org, Oct 27
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b21b82d90624320d13332e4eec413e4088295ce6

commit b21b82d90624320d13332e4eec413e4088295ce6
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Oct 27 12:58:24 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1901 1905 1906 1908 1909 1911 1913 1917 1919 1920 1927 1928 1929

1 lines were removed by consecutive passes since 1885.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: If76abd00c48e0a4013adb44b0206a9eb5fb11235
Reviewed-on: https://chromium-review.googlesource.com/741304
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512160}
[modify] https://crrev.com/b21b82d90624320d13332e4eec413e4088295ce6/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 127 by bugdroid1@chromium.org, Oct 28
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/65d54f1e7b7d7160c403db1711cba7b102e73e7f

commit 65d54f1e7b7d7160c403db1711cba7b102e73e7f
Author: Koji Ishii <kojii@chromium.org>
Date: Sat Oct 28 07:55:18 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1933 1934 1935 1938 1939

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ibdc98e8e279f2a7b9e304c073db3f5cfc636a491
Reviewed-on: https://chromium-review.googlesource.com/742821
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512392}
[modify] https://crrev.com/65d54f1e7b7d7160c403db1711cba7b102e73e7f/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 128 by bugdroid1@chromium.org, Nov 2
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3017c3bdd298e84f7730206f4e894319c1662c0e

commit 3017c3bdd298e84f7730206f4e894319c1662c0e
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Nov 02 10:08:38 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1941 1943 1945 1948 1949 1950 1953 1960 1967 1968 1970 1974 1975 1976
1977 1981 1982 1990 1992 1993

108 lines were removed by consecutive passes since 1960.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ice8162aa3fd3f4d23291f3dff063ece0bb438c42
Reviewed-on: https://chromium-review.googlesource.com/750602
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#513445}
[modify] https://crrev.com/3017c3bdd298e84f7730206f4e894319c1662c0e/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 129 by bugdroid1@chromium.org, Nov 6
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b9753ec26e440ca6bffae5185c387f67ff7ee968

commit b9753ec26e440ca6bffae5185c387f67ff7ee968
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Nov 06 15:18:21 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
1994 1995 1997 1999 2000 2002 2003 2004 2005

253 lines were removed by consecutive passes since 1990.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: If6b5399150254203504e4bb2cc756b1b3b55c92f
Reviewed-on: https://chromium-review.googlesource.com/753279
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#514132}
[modify] https://crrev.com/b9753ec26e440ca6bffae5185c387f67ff7ee968/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 130 by bugdroid1@chromium.org, Nov 7
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/626f2b2b3e927a369f6550cdc288d3a013fae0f1

commit 626f2b2b3e927a369f6550cdc288d3a013fae0f1
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Nov 07 12:49:27 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2006 2012 2015 2022

34 lines were removed by consecutive passes since 2002.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I2c670a280d3046674bb6c4886385155018045b30
Reviewed-on: https://chromium-review.googlesource.com/756775
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#514455}
[modify] https://crrev.com/626f2b2b3e927a369f6550cdc288d3a013fae0f1/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Blockedon: 783102
Project Member Comment 132 by bugdroid1@chromium.org, Nov 9
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/83c20aaa9d18e7a17a7342d8cf4a2f6c0687bb72

commit 83c20aaa9d18e7a17a7342d8cf4a2f6c0687bb72
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Nov 09 10:35:49 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2029 2033 2034 2037 2039 2047 2057 2061 2064

19 lines were removed by consecutive passes since 2012.

The following test sometimes pass in LayoutNG but we can't mark
it to pass when it has failing expectations. The expectation was
removed and marked as failure in TestExpectations instead.

- external/wpt/css/css-flexbox/percentage-heights-003.html

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ie6f035e28669ba61915e8783b807e8a6adb74276
Reviewed-on: https://chromium-review.googlesource.com/760040
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515131}
[modify] https://crrev.com/83c20aaa9d18e7a17a7342d8cf4a2f6c0687bb72/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[modify] https://crrev.com/83c20aaa9d18e7a17a7342d8cf4a2f6c0687bb72/third_party/WebKit/LayoutTests/TestExpectations
[delete] https://crrev.com/191b2267b37903d3bda773716c9a28214db1fca7/third_party/WebKit/LayoutTests/external/wpt/css/css-flexbox/percentage-heights-003-expected.txt

Project Member Comment 133 by bugdroid1@chromium.org, Nov 12 (6 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b5aafcb6bd1e539c4f7832c4428a4c4dfb7f6cdd

commit b5aafcb6bd1e539c4f7832c4428a4c4dfb7f6cdd
Author: Koji Ishii <kojii@chromium.org>
Date: Sun Nov 12 19:33:07 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2065 2066 2067 2069 2070 2071 2072 2078 2080 2082 2085

14 lines were removed by consecutive passes since 2057.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I7272d2dc468d7bc968d74f49bd5a2df94ef6ea4c
Reviewed-on: https://chromium-review.googlesource.com/765547
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515869}
[modify] https://crrev.com/b5aafcb6bd1e539c4f7832c4428a4c4dfb7f6cdd/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 134 by bugdroid1@chromium.org, Nov 13 (5 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9f5df28912f4119969b747b7675531df1b8dd870

commit 9f5df28912f4119969b747b7675531df1b8dd870
Author: Yoshifumi Inoue <yosin@chromium.org>
Date: Mon Nov 13 03:30:30 2017

Introduce static assert in LayoutMixIn to check class parameter is based on LayoutBlockFlow

This patch introduces static assert to check class parameter of |LayoutMixin<T>|
is |LayoutBlockFlow| or derived class to avoid misuse of |LayoutMixin<T>| for improving code
health.

Bug: 591099
Change-Id: I252682e62b7a23d7e8af6cdef45b6810f5750780
Reviewed-on: https://chromium-review.googlesource.com/763028
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515880}
[modify] https://crrev.com/9f5df28912f4119969b747b7675531df1b8dd870/third_party/WebKit/Source/core/layout/ng/layout_ng_mixin.h

Project Member Comment 135 by bugdroid1@chromium.org, Nov 13 (5 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0fb39b908b7d7e7bdd3d03045dbe8ea4d00dacee

commit 0fb39b908b7d7e7bdd3d03045dbe8ea4d00dacee
Author: Ian Kilpatrick <ikilpatrick@chromium.org>
Date: Mon Nov 13 04:19:17 2017

[LayoutNG] Update TestExpectations.

Bug: 591099
Change-Id: I8c006a0669e056c9a779a282e30cbea6ae106eb4
Reviewed-on: https://chromium-review.googlesource.com/763581
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515884}
[modify] https://crrev.com/0fb39b908b7d7e7bdd3d03045dbe8ea4d00dacee/third_party/WebKit/LayoutTests/TestExpectations

Project Member Comment 136 by bugdroid1@chromium.org, Nov 13 (5 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f4c602e7b9472db0a8bce5d27d2835af92bd78d1

commit f4c602e7b9472db0a8bce5d27d2835af92bd78d1
Author: Koji Ishii <kojii@chromium.org>
Date: Mon Nov 13 15:51:09 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2086 2087 2088 2090 2091 2092 2094 2095 2097

85 lines were removed by consecutive passes since 2069.

Also, 7525 lines were "fully deflaked" by using last 38 bot results
since 2000. It means all expectations including Pass are removed if
they were not seen in the last 38 results. Many of them might have
been added in CL:750602 probably due to bots config errors. This
may cause some flakiness for a while though.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ifc3da8c4b40d2e5024b86a9d4aa053bbd322d992
Reviewed-on: https://chromium-review.googlesource.com/766148
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515949}
[modify] https://crrev.com/f4c602e7b9472db0a8bce5d27d2835af92bd78d1/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 137 by bugdroid1@chromium.org, Nov 14 (4 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ca5a215318634967ce9aed09a652e48e57b9cfc6

commit ca5a215318634967ce9aed09a652e48e57b9cfc6
Author: Koji Ishii <kojii@chromium.org>
Date: Tue Nov 14 07:17:10 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2098 2100 2101 2106 2108 2110 2112 2115

Did not run de-flake because the last CL was full-deflake,
and currently bots look like a bit flaky.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I37b87d8ce98b9eb499361ceb07cb6491e5c0359d
Reviewed-on: https://chromium-review.googlesource.com/768189
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516221}
[modify] https://crrev.com/ca5a215318634967ce9aed09a652e48e57b9cfc6/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 138 by bugdroid1@chromium.org, Nov 14 (4 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d301cfc6b7aaf6b88738e16040534b5e8358aca2

commit d301cfc6b7aaf6b88738e16040534b5e8358aca2
Author: Yoshifumi Inoue <yosin@chromium.org>
Date: Tue Nov 14 09:01:22 2017

[LayoutNG] Introduce CanUseInlineBox() for tracking project progress

This patch introduces |CanUseInlineBox()| as a preparation to add
|DCHECK(CanUseInlineBox())| to functions using |InlineBox| for marker of
progress of LayoutNG project. To release LayoutNG, we should not hit
these |DCHECK()|.

Following patch will utilize |CanUseInlineBox()| into layout/ and editing/.

The patch[1] is a sample of utilization.

[1] http://crrev.com/c/763188 [LayoutNG] Utilize DCHECK(CanUseInlineBox()) in
core/layout/


Bug: 591099, 707656
Change-Id: Id71c39ae8520ab18fd1b8b8f3e18a66e8a7d567c
Reviewed-on: https://chromium-review.googlesource.com/768572
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516239}
[modify] https://crrev.com/d301cfc6b7aaf6b88738e16040534b5e8358aca2/third_party/WebKit/Source/core/layout/line/InlineBox.cpp
[modify] https://crrev.com/d301cfc6b7aaf6b88738e16040534b5e8358aca2/third_party/WebKit/Source/core/layout/line/InlineBox.h

Project Member Comment 139 by bugdroid1@chromium.org, Nov 15 (3 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1ca3acf50cf74c8532e604a66e7d408baf114e92

commit 1ca3acf50cf74c8532e604a66e7d408baf114e92
Author: Xiaocheng Hu <xiaochengh@chromium.org>
Date: Wed Nov 15 21:06:57 2017

[LayoutNG] Add DHECK of CanUseInlineBox() in ContainerNode.cpp

ContainerNode::GetUpperLeftCorner() uses legacy inline structures, which
will be removed when LayoutNG is enabled. This patch adds a DCHECK to
ease the tracking of the conversion of existing usage of legacy inline
structures.

Bug: 591099
Change-Id: Idac4b13a590e9799affe1f5a34726ecc7f7f1bab
Reviewed-on: https://chromium-review.googlesource.com/769567
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516824}
[modify] https://crrev.com/1ca3acf50cf74c8532e604a66e7d408baf114e92/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/1ca3acf50cf74c8532e604a66e7d408baf114e92/third_party/WebKit/Source/core/dom/ContainerNode.cpp

Project Member Comment 140 by bugdroid1@chromium.org, Nov 16 (2 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b7340f3abb7a61e346d2a5ad92dbe7e4a7a69fc7

commit b7340f3abb7a61e346d2a5ad92dbe7e4a7a69fc7
Author: Yoshifumi Inoue <yosin@chromium.org>
Date: Thu Nov 16 06:33:40 2017

[LayoutNG] LayoutInline::DirtyLinesFromChangedChild()

This patch makes |LayoutInline::DirtyLinesFromChangedChild()| to work with
LayoutNG by marking layout dirty for |LayoutInline| node instead of marking
|InlineBox| for specified child node because LayoutNG doesn't support
partial dirtiness of box fragment yet.

Bug: 591099
Change-Id: I1f0b616f1fed891e072a84df74a66fe324c85df2
Reviewed-on: https://chromium-review.googlesource.com/768257
Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516998}
[modify] https://crrev.com/b7340f3abb7a61e346d2a5ad92dbe7e4a7a69fc7/third_party/WebKit/Source/core/layout/LayoutInline.cpp
[modify] https://crrev.com/b7340f3abb7a61e346d2a5ad92dbe7e4a7a69fc7/third_party/WebKit/Source/core/layout/LayoutInline.h

Comment 141 by yosin@chromium.org, Nov 16 (2 days ago)
Blockedon: 785687
Project Member Comment 142 by bugdroid1@chromium.org, Nov 16 (2 days ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/df248dda4bdba7cda6e90befc5348dccfaba34fd

commit df248dda4bdba7cda6e90befc5348dccfaba34fd
Author: Koji Ishii <kojii@chromium.org>
Date: Thu Nov 16 16:10:20 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2116 2123 2127 2133 2137 2139 2140 2145 2146 2147 2149 2150 2151 2153
2154 2155 2159 2165 2166 2168

282 lines were removed by consecutive passes since 2098.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: I120acfceac4e9e556c039250db5e90cd5c087ce5
Reviewed-on: https://chromium-review.googlesource.com/774918
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517098}
[modify] https://crrev.com/df248dda4bdba7cda6e90befc5348dccfaba34fd/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Project Member Comment 143 by bugdroid1@chromium.org, Yesterday (27 hours ago)
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d8cc16b122b3836aaaa7cf71d560c134de72a213

commit d8cc16b122b3836aaaa7cf71d560c134de72a213
Author: Koji Ishii <kojii@chromium.org>
Date: Fri Nov 17 21:33:14 2017

[LayoutNG] Update FlagExpectations for LayoutNG

Following bot results are included.
2170 2172 2173 2174 2176 2181 2183 2184 2193 2194 2196

35 lines were removed by consecutive passes since 2154.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Change-Id: Ia76855815ba3b2abf85d16be12bd9044bb0d5a11
Reviewed-on: https://chromium-review.googlesource.com/777025
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517561}
[modify] https://crrev.com/d8cc16b122b3836aaaa7cf71d560c134de72a213/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG

Sign in to add a comment