Reland of Use NGLogicalRect instead of NGExclusion for exclusions.

Reland of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #1 id:1 of https://codereview.chromium.org/2467803002/ )

Reason for revert:
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).

Original issue's description:
> 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}

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/33a189c87461dcecd60c7090e98aeb4a8f923964
Cr-Commit-Position: refs/heads/master@{#428998}

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

The CQ bit was checked by kjellander@chromium.org

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

Created Reland of Use NGLogicalRect instead of NGExclusion for exclusions.

[commit-bot: I haz the power, 2016-11-01 14:26:51]

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 14:28:04]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Reland of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #1
id:1 of https://codereview.chromium.org/2467803002/ )

Reason for revert:
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).

Original issue's description:
> 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}

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

==========
Reland of Use NGLogicalRect instead of NGExclusion for exclusions. (patchset #1
id:1 of https://codereview.chromium.org/2467803002/ )

Reason for revert:
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).

Original issue's description:
> 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}

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/33a189c87461dcecd60c7090e98aeb4a8f923964
Cr-Commit-Position: refs/heads/master@{#428998}
==========

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

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

[eae, 2016-11-01 16:28:32]

LGTM