binding: Sets location[Symbol.toPrimitive].

binding: Sets location[Symbol.toPrimitive].

Let the location objects have Symbol.toPrimitive property
as an unconfiguable own property, so that author scripts
cannot inject nor overwrite it.

BUG=680409
Review-Url: https://codereview.chromium.org/2656973002
Cr-Commit-Position: refs/heads/master@{#446991}
Committed: https://chromium.googlesource.com/chromium/src/+/1f959adce3efa86d7d9560f9642c6ae816f8d81c

[Yuki, 2017-01-26 15:15:04]

Description was changed from

==========
binding: Sets location[Symbol.toPrimitive].

BUG=
==========

to

==========
binding: Sets location[Symbol.toPrimitive].

BUG=680409
==========

[Yuki, 2017-01-30 08:04:05]

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

[commit-bot: I haz the power, 2017-01-30 08:04:18]

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

[Yuki, 2017-01-30 08:08:12]

Description was changed from

==========
binding: Sets location[Symbol.toPrimitive].

BUG=680409
==========

to

==========
binding: Sets location[Symbol.toPrimitive].

Let the location objects have Symbol.toPrimitive property
as an unconfiguable own property, so that author scripts
cannot inject nor overwrite it.

BUG=680409
==========

[Yuki, 2017-01-30 08:35:38]

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

[Yuki, 2017-01-30 08:35:39]

Could you review this CL?

[haraken, 2017-01-30 09:36:42]

LGTM

Can we add a test?

[commit-bot: I haz the power, 2017-01-30 11:17:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-30 11:17:29]

Dry run: This issue passed the CQ dry run.

[Yuki, 2017-01-30 14:04:56]

Agreed.  Added a layout test.

[Yuki, 2017-01-30 14:05:08]

The CQ bit was checked by yukishiino@chromium.org

[Yuki, 2017-01-30 14:05:08]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2656973002/#ps40001
(title: "Added a layout test.")

[commit-bot: I haz the power, 2017-01-30 14:05:18]

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-01-30 16:06:17]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485785108738690,
"parent_rev": "bad28af4b6122f371e3ed467a8d5a9443624e751", "commit_rev":
"1f959adce3efa86d7d9560f9642c6ae816f8d81c"}

[commit-bot: I haz the power, 2017-01-30 16:06:52]

Description was changed from

==========
binding: Sets location[Symbol.toPrimitive].

Let the location objects have Symbol.toPrimitive property
as an unconfiguable own property, so that author scripts
cannot inject nor overwrite it.

BUG=680409
==========

to

==========
binding: Sets location[Symbol.toPrimitive].

Let the location objects have Symbol.toPrimitive property
as an unconfiguable own property, so that author scripts
cannot inject nor overwrite it.

BUG=680409

Review-Url: https://codereview.chromium.org/2656973002
Cr-Commit-Position: refs/heads/master@{#446991}
Committed:
https://chromium.googlesource.com/chromium/src/+/1f959adce3efa86d7d9560f9642c...
==========

[commit-bot: I haz the power, 2017-01-30 16:06:53]

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