Fix missed invalidation in markFixedPositionObjectForLayoutIfNeeded.

Fix missed invalidation in markFixedPositionObjectForLayoutIfNeeded.

If a fixed-position element had "auto" offsets in both block and inline flow
directions, the LayoutOnlyFixedPositionedObjects codepath failed to mark it for
layout when its computed logical top changed.

BUG=674867
Review-Url: https://codereview.chromium.org/2683773004
Cr-Commit-Position: refs/heads/master@{#448889}
Committed: https://chromium.googlesource.com/chromium/src/+/ccf2862c092b2339e42d7131d219b18060a773a7

[skobes, 2017-02-08 01:13:17]

The CQ bit was checked by skobes@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-08 01:13:56]

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

[skobes, 2017-02-08 01:15:51]

Description was changed from

==========
Fix missed invalidation in markFixedPositionObjectForLayoutIfNeeded.

If a fixed-position element had "auto" left, top, width, and height, then the
LayoutOnlyFixedPositionedObjects codepath failed to mark it for layout when its
computed top changed.

The hasStaticBlockPosition and hasStaticInlinePosition methods of ComputedStyle
are confusingly named - they do not imply position: static, but rather that the
CSS offset properties are "auto" in the block flow / inline flow directions.

BUG=674867
==========

to

==========
Fix missed invalidation in markFixedPositionObjectForLayoutIfNeeded.

If a fixed-position element had "auto" offsets in both block and inline flow
directions, the LayoutOnlyFixedPositionedObjects codepath failed to mark it for
layout when its computed logical top changed.

BUG=674867
==========

[skobes, 2017-02-08 01:19:31]

skobes@chromium.org changed reviewers:
+ szager@chromium.org

[szager1, 2017-02-08 01:23:45]

gross

lgtm

[skobes, 2017-02-08 01:25:39]

The CQ bit was unchecked by skobes@chromium.org

[skobes, 2017-02-08 01:25:53]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2017-02-08 01:26:01]

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-02-08 03:17:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486517153202880, "parent_rev":
"48ad1681cf6efac487c260d75ac5d5f35ab493e3", "commit_rev":
"ccf2862c092b2339e42d7131d219b18060a773a7"}

[commit-bot: I haz the power, 2017-02-08 03:18:35]

Description was changed from

==========
Fix missed invalidation in markFixedPositionObjectForLayoutIfNeeded.

If a fixed-position element had "auto" offsets in both block and inline flow
directions, the LayoutOnlyFixedPositionedObjects codepath failed to mark it for
layout when its computed logical top changed.

BUG=674867
==========

to

==========
Fix missed invalidation in markFixedPositionObjectForLayoutIfNeeded.

If a fixed-position element had "auto" offsets in both block and inline flow
directions, the LayoutOnlyFixedPositionedObjects codepath failed to mark it for
layout when its computed logical top changed.

BUG=674867

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

[commit-bot: I haz the power, 2017-02-08 03:18:36]

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