Disable tests to fix contextual property access

Disable tests to fix contextual property access.

This is required to land https://chromium-review.googlesource.com/c/483199/ which changes the behavior. Currently those tests hardwire wrong behavior that exposes internal implementation details. The change in the referred CL makes it more uniform, and in line with at least Firefox.
 

BUG=chromium:713732
Review-Url: https://codereview.chromium.org/2831923003
Cr-Commit-Position: refs/heads/master@{#467965}
Committed: https://chromium.googlesource.com/chromium/src/+/50d4cffe6fc36e4852ec2cb77a33e93d7beb5a51

[Toon Verwaest, 2017-04-26 11:55:36]

verwaest@chromium.org changed reviewers:
+ haraken@chromium.org

[Toon Verwaest, 2017-04-26 11:55:36]

ptal

[haraken, 2017-04-26 12:09:08]

LGTM (I discussed offline with Toon.)

- Would you add more explanation to the CL description?
- And ask kinuko@ if the expectation change on the service worker tests is okay?

[Toon Verwaest, 2017-04-26 12:14:05]

Description was changed from

==========
Disable tests to fix contextual property access

BUG=chromium:713732
==========

to

==========
Disable tests to fix contextual property access.

This is required to land https://chromium-review.googlesource.com/c/483199/
which changes the behavior. Currently those tests hardwire wrong behavior that
exposes internal implementation details. The change in the referred CL makes it
more uniform, and in line with at least Firefox.
 

BUG=chromium:713732
==========

[Toon Verwaest, 2017-04-26 12:14:38]

Added a comment. I've already asked on the linked bug a week ago, but didn't get
a response. I've now pinged Kinuko via chat, so let's see.

[kinuko, 2017-04-28 06:02:46]

From the bug discussion I believe this (and the actual change) is fine. We'll
revise the test case. LGTM

[Toon Verwaest, 2017-04-28 09:17:12]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2017-04-28 09:17:34]

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-04-28 12:10:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493371032221650, "parent_rev":
"cce7ad62c6fc4ee4c7cd1b32c2f1bb77c5f66dc0", "commit_rev":
"50d4cffe6fc36e4852ec2cb77a33e93d7beb5a51"}

[commit-bot: I haz the power, 2017-04-28 12:10:21]

Description was changed from

==========
Disable tests to fix contextual property access.

This is required to land https://chromium-review.googlesource.com/c/483199/
which changes the behavior. Currently those tests hardwire wrong behavior that
exposes internal implementation details. The change in the referred CL makes it
more uniform, and in line with at least Firefox.
 

BUG=chromium:713732
==========

to

==========
Disable tests to fix contextual property access.

This is required to land https://chromium-review.googlesource.com/c/483199/
which changes the behavior. Currently those tests hardwire wrong behavior that
exposes internal implementation details. The change in the referred CL makes it
more uniform, and in line with at least Firefox.
 

BUG=chromium:713732

Review-Url: https://codereview.chromium.org/2831923003
Cr-Commit-Position: refs/heads/master@{#467965}
Committed:
https://chromium.googlesource.com/chromium/src/+/50d4cffe6fc36e4852ec2cb77a33...
==========

[commit-bot: I haz the power, 2017-04-28 12:10:22]

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