Revert of Use NGLogicalRect instead of NGExclusion for exclusions.

Revert of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #5 id:110016 of https://codereview.chromium.org/2457013004/ )

Reason for revert:
Reverting to enable reverting of https://codereview.chromium.org/2458253002/ in https://codereview.chromium.org/2458253002/ due to  merge conflicts.

Please reland once the flakiness has been sorted out.

Original issue's description:
> Use NGLogicalRect instead of NGExclusion to work with exclusions.
>
> This patch does the following:
> - Delete NGExclusion
> - Change the code to use NGLogicalRect to store exclusions
> - Make NGPhysicalConstraintSpace to own the list of exclusions through Vector<std::unique_ptr>
> - Introduce WRITING_MODE_IGNORED macros that can be used to annotate cases where the writing mode is inored
>
> BUG=635619
>
> Committed: https://crrev.com/250a6979a3311cb8a39ca7d7f316353550362f86
> Cr-Commit-Position: refs/heads/master@{#428907}

TBR=cbiesinger@chromium.org,eae@chromium.org,ikilpatrick@chromium.org,dpranke@chromium.org,glebl@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=635619
Committed: https://crrev.com/3b41bfed7efb2d172f8a9706f09c0b793adda997
Cr-Commit-Position: refs/heads/master@{#428971}

[kjellander_chromium, 2016-11-01 10:29:55]

The CQ bit was checked by kjellander@chromium.org

[kjellander_chromium, 2016-11-01 10:29:55]

Created Revert of Use NGLogicalRect instead of NGExclusion for exclusions.

[commit-bot: I haz the power, 2016-11-01 10:30:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-01 10:31:16]

Description was changed from

==========
Revert of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #5
id:110016 of https://codereview.chromium.org/2457013004/ )

Reason for revert:
Reverting to enable reverting of https://codereview.chromium.org/2458253002/ in
https://codereview.chromium.org/2458253002/ due to  merge conflicts.

Please reland once the flakiness has been sorted out.

Original issue's description:
> Use NGLogicalRect instead of NGExclusion to work with exclusions.
> 
> This patch does the following:
> - Delete NGExclusion
> - Change the code to use NGLogicalRect to store exclusions
> - Make NGPhysicalConstraintSpace to own the list of exclusions through
Vector<std::unique_ptr>
> - Introduce WRITING_MODE_IGNORED macros that can be used to annotate cases
where the writing mode is inored
> 
> BUG=635619
> 
> Committed: https://crrev.com/250a6979a3311cb8a39ca7d7f316353550362f86
> Cr-Commit-Position: refs/heads/master@{#428907}

TBR=cbiesinger@chromium.org,eae@chromium.org,ikilpatrick@chromium.org,dpranke...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=635619
==========

to

==========
Revert of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #5
id:110016 of https://codereview.chromium.org/2457013004/ )

Reason for revert:
Reverting to enable reverting of https://codereview.chromium.org/2458253002/ in
https://codereview.chromium.org/2458253002/ due to  merge conflicts.

Please reland once the flakiness has been sorted out.

Original issue's description:
> Use NGLogicalRect instead of NGExclusion to work with exclusions.
> 
> This patch does the following:
> - Delete NGExclusion
> - Change the code to use NGLogicalRect to store exclusions
> - Make NGPhysicalConstraintSpace to own the list of exclusions through
Vector<std::unique_ptr>
> - Introduce WRITING_MODE_IGNORED macros that can be used to annotate cases
where the writing mode is inored
> 
> BUG=635619
> 
> Committed: https://crrev.com/250a6979a3311cb8a39ca7d7f316353550362f86
> Cr-Commit-Position: refs/heads/master@{#428907}

TBR=cbiesinger@chromium.org,eae@chromium.org,ikilpatrick@chromium.org,dpranke...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=635619
==========

[commit-bot: I haz the power, 2016-11-01 10:31:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-01 10:33:24]

Description was changed from

==========
Revert of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #5
id:110016 of https://codereview.chromium.org/2457013004/ )

Reason for revert:
Reverting to enable reverting of https://codereview.chromium.org/2458253002/ in
https://codereview.chromium.org/2458253002/ due to  merge conflicts.

Please reland once the flakiness has been sorted out.

Original issue's description:
> Use NGLogicalRect instead of NGExclusion to work with exclusions.
> 
> This patch does the following:
> - Delete NGExclusion
> - Change the code to use NGLogicalRect to store exclusions
> - Make NGPhysicalConstraintSpace to own the list of exclusions through
Vector<std::unique_ptr>
> - Introduce WRITING_MODE_IGNORED macros that can be used to annotate cases
where the writing mode is inored
> 
> BUG=635619
> 
> Committed: https://crrev.com/250a6979a3311cb8a39ca7d7f316353550362f86
> Cr-Commit-Position: refs/heads/master@{#428907}

TBR=cbiesinger@chromium.org,eae@chromium.org,ikilpatrick@chromium.org,dpranke...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=635619
==========

to

==========
Revert of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #5
id:110016 of https://codereview.chromium.org/2457013004/ )

Reason for revert:
Reverting to enable reverting of https://codereview.chromium.org/2458253002/ in
https://codereview.chromium.org/2458253002/ due to  merge conflicts.

Please reland once the flakiness has been sorted out.

Original issue's description:
> Use NGLogicalRect instead of NGExclusion to work with exclusions.
>
> This patch does the following:
> - Delete NGExclusion
> - Change the code to use NGLogicalRect to store exclusions
> - Make NGPhysicalConstraintSpace to own the list of exclusions through
Vector<std::unique_ptr>
> - Introduce WRITING_MODE_IGNORED macros that can be used to annotate cases
where the writing mode is inored
>
> BUG=635619
>
> Committed: https://crrev.com/250a6979a3311cb8a39ca7d7f316353550362f86
> Cr-Commit-Position: refs/heads/master@{#428907}

TBR=cbiesinger@chromium.org,eae@chromium.org,ikilpatrick@chromium.org,dpranke...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=635619

Committed: https://crrev.com/3b41bfed7efb2d172f8a9706f09c0b793adda997
Cr-Commit-Position: refs/heads/master@{#428971}
==========

[commit-bot: I haz the power, 2016-11-01 10:33:25]

Patchset 1 (id:??) landed as
https://crrev.com/3b41bfed7efb2d172f8a9706f09c0b793adda997
Cr-Commit-Position: refs/heads/master@{#428971}

[kjellander_chromium, 2016-11-01 14:26:38]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2462413002/ by kjellander@chromium.org.

The reason for reverting is: Relanding on top of
https://codereview.chromium.org/2466913002/ since that one wasn't the culprit
for the webkit_tests failures on WebKit Win7 (dbg)..