binding: Makes non-cross-origin-accessible attrs be accessor props.

binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

The previous attempt was http://crrev.com/1380503002

BUG=475556
Committed: https://crrev.com/5b2adfa4d4e2fb6b74d177625605dcece20a40b7
Cr-Commit-Position: refs/heads/master@{#434942}

[Yuki, 2016-11-18 11:54:30]

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

[commit-bot: I haz the power, 2016-11-18 11:54:38]

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

[Yuki, 2016-11-18 12:36:23]

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

[commit-bot: I haz the power, 2016-11-18 12:36:39]

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

[Yuki, 2016-11-18 14:31:09]

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

[Yuki, 2016-11-18 14:31:10]

Could you review this CL?

This is another try to make Window and Location's attributes be accessor
properties except for cross-origin-accessible attributes.

[commit-bot: I haz the power, 2016-11-18 14:33:43]

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

[commit-bot: I haz the power, 2016-11-18 14:33:44]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[haraken, 2016-11-18 15:53:44]

LGTM to try.

[Yuki, 2016-11-21 07:38:48]

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

[commit-bot: I haz the power, 2016-11-21 07:38:53]

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

[Yuki, 2016-11-21 07:41:37]

Description was changed from

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

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

to

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

BUG=475556
==========

[Yuki, 2016-11-21 07:52:07]

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

[commit-bot: I haz the power, 2016-11-21 07:52:18]

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

[Yuki, 2016-11-21 07:53:47]

Description was changed from

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

BUG=475556
==========

to

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

The previous attempt was http://crrev.com/1380503002

BUG=475556
==========

[commit-bot: I haz the power, 2016-11-21 09:43:02]

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

[commit-bot: I haz the power, 2016-11-21 09:43:02]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Yuki, 2016-11-21 11:57:48]

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

[commit-bot: I haz the power, 2016-11-21 11:57:53]

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

[Yuki, 2016-11-21 13:34:58]

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

[Yuki, 2016-11-21 13:34:59]

+R=tkent@,

Could you review as an owner of gloabl-interface-listing things?

We're going to make second attempt to change Window's attributes from
data-properties to accessor-properties.
(The first attempt was okay as a behavioral change, but we reverted it due to a
performance regression.)

[tkent, 2016-11-21 14:13:25]

lgtm

[commit-bot: I haz the power, 2016-11-21 14:43:35]

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

[commit-bot: I haz the power, 2016-11-21 14:43:36]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Yuki, 2016-11-22 10:23:44]

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

[commit-bot: I haz the power, 2016-11-22 10:23:58]

Dry run: 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-22 12:10:00]

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

[commit-bot: I haz the power, 2016-11-22 12:10:02]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Yuki, 2016-11-22 13:15:47]

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

[commit-bot: I haz the power, 2016-11-22 13:16:04]

Dry run: 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-22 16:00:28]

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

[commit-bot: I haz the power, 2016-11-22 16:00:29]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Yuki, 2016-11-24 09:10:58]

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

[commit-bot: I haz the power, 2016-11-24 09:11:25]

Dry run: 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-24 10:41:11]

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

[commit-bot: I haz the power, 2016-11-24 10:41:12]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Yuki, 2016-11-28 06:49:56]

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

[commit-bot: I haz the power, 2016-11-28 06:50:10]

Dry run: 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-28 08:22:22]

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

[commit-bot: I haz the power, 2016-11-28 08:22:23]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Yuki, 2016-11-28 08:53:05]

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

[commit-bot: I haz the power, 2016-11-28 08:53:18]

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

[Yuki, 2016-11-28 09:47:09]

The CQ bit was unchecked by yukishiino@chromium.org

[Yuki, 2016-11-28 09:47:18]

The CQ bit was checked by yukishiino@chromium.org

[Yuki, 2016-11-28 09:47:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2506393004/#ps180001
(title: "Updated a test expectation.")

[commit-bot: I haz the power, 2016-11-28 09:47: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, 2016-11-28 12:09:21]

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

[commit-bot: I haz the power, 2016-11-28 12:09:22]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Yuki, 2016-11-28 12:15:20]

The CQ bit was checked by yukishiino@chromium.org

[commit-bot: I haz the power, 2016-11-28 12:15:32]

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-28 15:08:48]

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

[commit-bot: I haz the power, 2016-11-28 15:08:50]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Yuki, 2016-11-29 05:05:28]

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

[commit-bot: I haz the power, 2016-11-29 05:05:45]

Dry run: 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-29 05:08:57]

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

[commit-bot: I haz the power, 2016-11-29 05:08:58]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Yuki, 2016-11-29 05:14:52]

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

[commit-bot: I haz the power, 2016-11-29 05:15:02]

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

[Yuki, 2016-11-29 08:30:57]

The CQ bit was unchecked by yukishiino@chromium.org

[Yuki, 2016-11-29 08:31:05]

The CQ bit was checked by yukishiino@chromium.org

[Yuki, 2016-11-29 08:31:05]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2506393004/#ps220001
(title: "Synced.")

[commit-bot: I haz the power, 2016-11-29 08:31:23]

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-29 09:51:46]

CQ is committing da patch.

Bot data: {"patchset_id": 220001, "attempt_start_ts": 1480408265138130,
"parent_rev": "6f4daf85297605c761f8450ee4b7654bb4152e35", "commit_rev":
"c3232d87fabac8e281790de6da5a68ac2148cc0e"}

[commit-bot: I haz the power, 2016-11-29 09:52:14]

Description was changed from

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

The previous attempt was http://crrev.com/1380503002

BUG=475556
==========

to

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

The previous attempt was http://crrev.com/1380503002

BUG=475556
==========

[commit-bot: I haz the power, 2016-11-29 09:52:16]

Committed patchset #12 (id:220001)

[commit-bot: I haz the power, 2016-11-29 09:54:08]

Description was changed from

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

The previous attempt was http://crrev.com/1380503002

BUG=475556
==========

to

==========
binding: Makes non-cross-origin-accessible attrs be accessor props.

Makes non-cross-origin accessible attributes be accessor
properties.

[CachedAccessor] already made window.document an accessor
property, and this CL doesn't affect it.

Cross-origin accessible attributes remain being data
properties.

The previous attempt was http://crrev.com/1380503002

BUG=475556

Committed: https://crrev.com/5b2adfa4d4e2fb6b74d177625605dcece20a40b7
Cr-Commit-Position: refs/heads/master@{#434942}
==========

[commit-bot: I haz the power, 2016-11-29 09:54:09]

Patchset 12 (id:??) landed as
https://crrev.com/5b2adfa4d4e2fb6b74d177625605dcece20a40b7
Cr-Commit-Position: refs/heads/master@{#434942}