Introduce Response.redirect attribute and add LayoutTest.

Introduce Response.redirected attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirected returns true when the size of the list is larger than 1.

BUG=658249
Committed: https://crrev.com/ca86b8af14cf71407b6e7ed342c8dbeb268dd51e
Cr-Commit-Position: refs/heads/master@{#437555}

[horo, 2016-11-22 09:08:24]

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

[commit-bot: I haz the power, 2016-11-22 09:08:46]

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 10:54:02]

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

[commit-bot: I haz the power, 2016-11-22 10:54:03]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[horo, 2016-11-24 06:05:10]

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

[commit-bot: I haz the power, 2016-11-24 06:05:26]

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

[horo, 2016-11-24 07:32:51]

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

[commit-bot: I haz the power, 2016-11-24 07:33:13]

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

[horo, 2016-11-24 08:46:15]

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

[commit-bot: I haz the power, 2016-11-24 08:46:22]

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:44:44]

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

[commit-bot: I haz the power, 2016-11-24 10:44:45]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[horo, 2016-11-25 06:28:37]

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

[commit-bot: I haz the power, 2016-11-25 06:28:43]

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-25 07:35:22]

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

[commit-bot: I haz the power, 2016-11-25 07:35: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_...)

[horo, 2016-11-25 08:41:57]

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

[commit-bot: I haz the power, 2016-11-25 08:42:11]

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-25 10:24:39]

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

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

Dry run: This issue passed the CQ dry run.

[horo, 2016-11-28 01:47:25]

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

[commit-bot: I haz the power, 2016-11-28 01:47:41]

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

[horo, 2016-11-28 01:48:45]

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

[commit-bot: I haz the power, 2016-11-28 01:48: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-28 04:30:03]

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

[commit-bot: I haz the power, 2016-11-28 04:30:03]

Dry run: This issue passed the CQ dry run.

[horo, 2016-11-29 06:52:54]

Patchset #7 (id:120001) has been deleted

[horo, 2016-11-29 06:52:57]

Patchset #6 (id:100001) has been deleted

[horo, 2016-11-29 06:53:00]

Patchset #5 (id:80001) has been deleted

[horo, 2016-11-29 06:53:03]

Patchset #4 (id:60001) has been deleted

[horo, 2016-11-29 06:53:06]

Patchset #3 (id:40001) has been deleted

[horo, 2016-11-29 06:53:07]

Patchset #2 (id:20001) has been deleted

[horo, 2016-11-29 06:53:10]

Patchset #1 (id:1) has been deleted

[horo, 2016-11-29 06:53:27]

Patchset #1 (id:140001) has been deleted

[horo, 2016-11-29 06:55:08]

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

[commit-bot: I haz the power, 2016-11-29 06:55:31]

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

[horo, 2016-11-29 07:25:08]

Description was changed from

==========
add .redirect and update LayoutTests

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

to

==========
Introduce Response.redirect attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirect returns true when the size of the list is larger than 1.

This URL list is an internal value which is not exposed to script.
To check the list in the LayoutTest, this CL introduces an internal API
getInternalResponseURLList().

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

[horo, 2016-11-29 07:25:23]

Description was changed from

==========
Introduce Response.redirect attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirect returns true when the size of the list is larger than 1.

This URL list is an internal value which is not exposed to script.
To check the list in the LayoutTest, this CL introduces an internal API
getInternalResponseURLList().

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

to

==========
Introduce Response.redirect attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirect returns true when the size of the list is larger than 1.

This URL list is an internal value which is not exposed to script.
To check the list in the LayoutTest, this CL introduces an internal API
getInternalResponseURLList().

BUG=658249
==========

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

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

[commit-bot: I haz the power, 2016-11-29 08:52:53]

Dry run: This issue passed the CQ dry run.

[horo, 2016-12-06 05:18:09]

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

[commit-bot: I haz the power, 2016-12-06 05:18:29]

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

[horo, 2016-12-06 05:19:36]

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

[commit-bot: I haz the power, 2016-12-06 05:19:47]

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

[horo, 2016-12-06 05:43:12]

Description was changed from

==========
Introduce Response.redirect attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirect returns true when the size of the list is larger than 1.

This URL list is an internal value which is not exposed to script.
To check the list in the LayoutTest, this CL introduces an internal API
getInternalResponseURLList().

BUG=658249
==========

to

==========
Introduce Response.redirect attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirect returns true when the size of the list is larger than 1.

BUG=658249
==========

[horo, 2016-12-06 05:43:33]

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

[commit-bot: I haz the power, 2016-12-06 05:43:47]

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-12-06 06:27:47]

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

[commit-bot: I haz the power, 2016-12-06 06:27:48]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[horo, 2016-12-06 10:29:23]

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

[commit-bot: I haz the power, 2016-12-06 10:29:37]

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-12-06 11:53:51]

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

[commit-bot: I haz the power, 2016-12-06 11:53:52]

Dry run: This issue passed the CQ dry run.

[horo, 2016-12-09 05:06:28]

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

[commit-bot: I haz the power, 2016-12-09 05:06:41]

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

[horo, 2016-12-09 05:55:30]

Patchset #4 (id:220001) has been deleted

[horo, 2016-12-09 05:55:33]

Patchset #3 (id:200001) has been deleted

[horo, 2016-12-09 05:55:47]

Patchset #2 (id:180001) has been deleted

[horo, 2016-12-09 05:55:59]

Patchset #1 (id:160001) has been deleted

[horo, 2016-12-09 06:28:46]

horo@chromium.org changed reviewers:
+ falken@chromium.org

[horo, 2016-12-09 06:28:47]

falken@
Could you please review this?

[falken, 2016-12-09 06:42:02]

lgtm

https://codereview.chromium.org/2524703002/diff/230001/third_party/WebKit/Lay...
File third_party/WebKit/LayoutTests/http/tests/fetch/resources/thorough-util.js
(right):

https://codereview.chromium.org/2524703002/diff/230001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/http/tests/fetch/resources/thorough-util.js:1:
var SCOPE = BASE_ORIGIN +
In CL description: Response.redirected not .redirect

[horo, 2016-12-09 08:18:14]

Description was changed from

==========
Introduce Response.redirect attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirect returns true when the size of the list is larger than 1.

BUG=658249
==========

to

==========
Introduce Response.redirected attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirected returns true when the size of the list is larger than 1.

BUG=658249
==========

[horo, 2016-12-09 08:19:29]

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

[horo, 2016-12-09 08:19:29]

tkent@
Could you please review global-interface-listing-expected.txt files?

[horo, 2016-12-09 08:19:54]

https://codereview.chromium.org/2524703002/diff/230001/third_party/WebKit/Lay...
File third_party/WebKit/LayoutTests/http/tests/fetch/resources/thorough-util.js
(right):

https://codereview.chromium.org/2524703002/diff/230001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/http/tests/fetch/resources/thorough-util.js:1:
var SCOPE = BASE_ORIGIN +
On 2016/12/09 06:42:02, falken wrote:
> In CL description: Response.redirected not .redirect

Done.

[commit-bot: I haz the power, 2016-12-09 08:24:42]

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

[commit-bot: I haz the power, 2016-12-09 08:24:43]

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_...)

[tkent, 2016-12-09 08:27:07]

lgtm

[horo, 2016-12-09 14:57:25]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2016-12-09 14:57:39]

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-12-09 16:25:55]

CQ is committing da patch.

Bot data: {"patchset_id": 230001, "attempt_start_ts": 1481295445837940,
"parent_rev": "594cd1d0c345c275ba4d8a5e36ed71c87933a1ee", "commit_rev":
"5742ab9817b2e0775fa10905598e12bd33b61089"}

[commit-bot: I haz the power, 2016-12-09 16:26:25]

Description was changed from

==========
Introduce Response.redirected attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirected returns true when the size of the list is larger than 1.

BUG=658249
==========

to

==========
Introduce Response.redirected attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirected returns true when the size of the list is larger than 1.

BUG=658249

Review-Url: https://codereview.chromium.org/2524703002
==========

[commit-bot: I haz the power, 2016-12-09 16:26:26]

Committed patchset #1 (id:230001)

[commit-bot: I haz the power, 2016-12-09 16:29:10]

Description was changed from

==========
Introduce Response.redirected attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirected returns true when the size of the list is larger than 1.

BUG=658249

Review-Url: https://codereview.chromium.org/2524703002
==========

to

==========
Introduce Response.redirected attribute and add LayoutTest.

https://codereview.chromium.org/2516353002/ introduces URL list of Response.
Response.redirected returns true when the size of the list is larger than 1.

BUG=658249

Committed: https://crrev.com/ca86b8af14cf71407b6e7ed342c8dbeb268dd51e
Cr-Commit-Position: refs/heads/master@{#437555}
==========

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

Patchset 1 (id:??) landed as
https://crrev.com/ca86b8af14cf71407b6e7ed342c8dbeb268dd51e
Cr-Commit-Position: refs/heads/master@{#437555}