Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow each other

Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow each other (patchset #5 id:100001 of https://codereview.chromium.org/2733133002/ )

Reason for revert:
Broke downstream bot (see bug)

Original issue's description:
> Combine 2 exclusions in Layout Opportunity Tree if they shadow each other
> 
> This patch introduces the logic that tries to combine 2 exclusions if
> possible.
> We can combine 2 exclusions if
>  - they are adjoining to each other and have the same exclusion type
>  - the new exclusion shadows the old one.
>    That's because it's not allowed to position anything in the
>    shadowed area.
> 
> Example:
>  <div id="SS" style="float: left; height: 10px; width: 10px"></div>
>  <div id="BB" style="float: left; height: 20px; width: 20px"></div>
>  +----------------+
>  |SSBB
>  |**BB
>  We combine SS and BB exclusions including the shadowed area (**).
> 
> BUG=635619
> TESTS=TwoRightExclusionsShadowEachOther,
> TwoLeftExclusionsShadowEachOther
> 
> Review-Url: https://codereview.chromium.org/2733133002
> Cr-Commit-Position: refs/heads/master@{#455444}
> Committed: https://chromium.googlesource.com/chromium/src/+/5ccd9ec6998eee48bc73e7d8a992c4cccdb31899

TBR=ikilpatrick@chromium.org,cbiesinger@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, 699703
Review-Url: https://codereview.chromium.org/2732223007
Cr-Commit-Position: refs/heads/master@{#455541}
Committed: https://chromium.googlesource.com/chromium/src/+/ad9e2730b5bf41af6386f0526d5f901fb515258e

[agrieve, 2017-03-08 21:09:25]

Created Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow
each other

[agrieve, 2017-03-08 21:10:01]

Description was changed from

==========
Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow each
other (patchset #5 id:100001 of https://codereview.chromium.org/2733133002/ )

Reason for revert:
Broke downstream bot (see bug)

Original issue's description:
> Combine 2 exclusions in Layout Opportunity Tree if they shadow each other
> 
> This patch introduces the logic that tries to combine 2 exclusions if
> possible.
> We can combine 2 exclusions if
>  - they are adjoining to each other and have the same exclusion type
>  - the new exclusion shadows the old one.
>    That's because it's not allowed to position anything in the
>    shadowed area.
> 
> Example:
>  <div id="SS" style="float: left; height: 10px; width: 10px"></div>
>  <div id="BB" style="float: left; height: 20px; width: 20px"></div>
>  +----------------+
>  |SSBB
>  |**BB
>  We combine SS and BB exclusions including the shadowed area (**).
> 
> BUG=635619
> TESTS=TwoRightExclusionsShadowEachOther,
> TwoLeftExclusionsShadowEachOther
> 
> Review-Url: https://codereview.chromium.org/2733133002
> Cr-Commit-Position: refs/heads/master@{#455444}
> Committed:
https://chromium.googlesource.com/chromium/src/+/5ccd9ec6998eee48bc73e7d8a992...

TBR=ikilpatrick@chromium.org,cbiesinger@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
==========

to

==========
Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow each
other (patchset #5 id:100001 of https://codereview.chromium.org/2733133002/ )

Reason for revert:
Broke downstream bot (see bug)

Original issue's description:
> Combine 2 exclusions in Layout Opportunity Tree if they shadow each other
> 
> This patch introduces the logic that tries to combine 2 exclusions if
> possible.
> We can combine 2 exclusions if
>  - they are adjoining to each other and have the same exclusion type
>  - the new exclusion shadows the old one.
>    That's because it's not allowed to position anything in the
>    shadowed area.
> 
> Example:
>  <div id="SS" style="float: left; height: 10px; width: 10px"></div>
>  <div id="BB" style="float: left; height: 20px; width: 20px"></div>
>  +----------------+
>  |SSBB
>  |**BB
>  We combine SS and BB exclusions including the shadowed area (**).
> 
> BUG=635619
> TESTS=TwoRightExclusionsShadowEachOther,
> TwoLeftExclusionsShadowEachOther
> 
> Review-Url: https://codereview.chromium.org/2733133002
> Cr-Commit-Position: refs/heads/master@{#455444}
> Committed:
https://chromium.googlesource.com/chromium/src/+/5ccd9ec6998eee48bc73e7d8a992...

TBR=ikilpatrick@chromium.org,cbiesinger@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,699703
==========

[agrieve, 2017-03-08 21:10:19]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2017-03-08 21:10:41]

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, 2017-03-08 21:11:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489007419629410, "parent_rev":
"4b85b58e36ed9f14bfda0ca84c82f5bdfb509ab7", "commit_rev":
"ad9e2730b5bf41af6386f0526d5f901fb515258e"}

[commit-bot: I haz the power, 2017-03-08 21:13:26]

Description was changed from

==========
Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow each
other (patchset #5 id:100001 of https://codereview.chromium.org/2733133002/ )

Reason for revert:
Broke downstream bot (see bug)

Original issue's description:
> Combine 2 exclusions in Layout Opportunity Tree if they shadow each other
> 
> This patch introduces the logic that tries to combine 2 exclusions if
> possible.
> We can combine 2 exclusions if
>  - they are adjoining to each other and have the same exclusion type
>  - the new exclusion shadows the old one.
>    That's because it's not allowed to position anything in the
>    shadowed area.
> 
> Example:
>  <div id="SS" style="float: left; height: 10px; width: 10px"></div>
>  <div id="BB" style="float: left; height: 20px; width: 20px"></div>
>  +----------------+
>  |SSBB
>  |**BB
>  We combine SS and BB exclusions including the shadowed area (**).
> 
> BUG=635619
> TESTS=TwoRightExclusionsShadowEachOther,
> TwoLeftExclusionsShadowEachOther
> 
> Review-Url: https://codereview.chromium.org/2733133002
> Cr-Commit-Position: refs/heads/master@{#455444}
> Committed:
https://chromium.googlesource.com/chromium/src/+/5ccd9ec6998eee48bc73e7d8a992...

TBR=ikilpatrick@chromium.org,cbiesinger@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,699703
==========

to

==========
Revert of Combine 2 exclusions in Layout Opportunity Tree if they shadow each
other (patchset #5 id:100001 of https://codereview.chromium.org/2733133002/ )

Reason for revert:
Broke downstream bot (see bug)

Original issue's description:
> Combine 2 exclusions in Layout Opportunity Tree if they shadow each other
> 
> This patch introduces the logic that tries to combine 2 exclusions if
> possible.
> We can combine 2 exclusions if
>  - they are adjoining to each other and have the same exclusion type
>  - the new exclusion shadows the old one.
>    That's because it's not allowed to position anything in the
>    shadowed area.
> 
> Example:
>  <div id="SS" style="float: left; height: 10px; width: 10px"></div>
>  <div id="BB" style="float: left; height: 20px; width: 20px"></div>
>  +----------------+
>  |SSBB
>  |**BB
>  We combine SS and BB exclusions including the shadowed area (**).
> 
> BUG=635619
> TESTS=TwoRightExclusionsShadowEachOther,
> TwoLeftExclusionsShadowEachOther
> 
> Review-Url: https://codereview.chromium.org/2733133002
> Cr-Commit-Position: refs/heads/master@{#455444}
> Committed:
https://chromium.googlesource.com/chromium/src/+/5ccd9ec6998eee48bc73e7d8a992...

TBR=ikilpatrick@chromium.org,cbiesinger@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,699703

Review-Url: https://codereview.chromium.org/2732223007
Cr-Commit-Position: refs/heads/master@{#455541}
Committed:
https://chromium.googlesource.com/chromium/src/+/ad9e2730b5bf41af6386f0526d5f...
==========

[commit-bot: I haz the power, 2017-03-08 21:13:27]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ad9e2730b5bf41af6386f0526d5f...