[Devtools][Code health] Changed NetworkRequest.url to proper getter/setter

[Devtools][Code health] Changed NetworkRequest.url to proper getter/setter

This patch moves NetworkRequest.prototype.url to a proper getter/setter
function to be within our styleguide.

R=caseq,luoe,einbinder,dgozman
BUG=679105
Review-Url: https://codereview.chromium.org/2626553002
Cr-Commit-Position: refs/heads/master@{#442736}
Committed: https://chromium.googlesource.com/chromium/src/+/8a062852968b7a329e313ab789bc69856ad8af29

[allada, 2017-01-09 22:46:02]

PTL

[dgozman, 2017-01-10 00:07:26]

lgtm

https://codereview.chromium.org/2626553002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/inspector/extensions-network-redirect.html
(right):

https://codereview.chromium.org/2626553002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/extensions-network-redirect.html:11:
debugger;
Debugger won't help you!

[allada, 2017-01-10 00:10:22]

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

[allada, 2017-01-10 00:10:31]

done.

https://codereview.chromium.org/2626553002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/inspector/extensions-network-redirect.html
(right):

https://codereview.chromium.org/2626553002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/extensions-network-redirect.html:11:
debugger;
On 2017/01/10 00:07:26, dgozman wrote:
> Debugger won't help you!

Done.

[commit-bot: I haz the power, 2017-01-10 00:11:08]

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

[allada, 2017-01-10 00:38:33]

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

[commit-bot: I haz the power, 2017-01-10 00:39:51]

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, 2017-01-10 02:18:07]

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

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

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

[allada, 2017-01-10 02:44:37]

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

[allada, 2017-01-10 02:45:15]

caseq@ or dgozman@ can you look at Test.js to make sure I did not overlook
anything? I renamed resource to request where appropriate.

[commit-bot: I haz the power, 2017-01-10 02:45:39]

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, 2017-01-10 04:46:54]

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

[commit-bot: I haz the power, 2017-01-10 04:46:54]

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

[allada, 2017-01-10 06:55:26]

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

[commit-bot: I haz the power, 2017-01-10 06:55:51]

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, 2017-01-10 09:03:42]

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

[commit-bot: I haz the power, 2017-01-10 09:03:43]

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

[allada, 2017-01-10 18:57:40]

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

[commit-bot: I haz the power, 2017-01-10 18:58:21]

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, 2017-01-10 20:58:42]

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

[commit-bot: I haz the power, 2017-01-10 20:58:43]

Dry run: This issue passed the CQ dry run.

[allada, 2017-01-10 23:50:57]

The CQ bit was checked by allada@chromium.org

[allada, 2017-01-10 23:50:58]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2626553002/#ps100001
(title: "changes")

[commit-bot: I haz the power, 2017-01-10 23:51:46]

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-10 23:58:16]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1484092257987450,
"parent_rev": "41afe676df04303cb9b0f67e4765b150969d3899", "commit_rev":
"8a062852968b7a329e313ab789bc69856ad8af29"}

[commit-bot: I haz the power, 2017-01-10 23:58:44]

Description was changed from

==========
[Devtools][Code health] Changed NetworkRequest.url to proper getter/setter

This patch moves NetworkRequest.prototype.url to a proper getter/setter
function to be within our styleguide.

R=caseq,luoe,einbinder,dgozman
BUG=679105
==========

to

==========
[Devtools][Code health] Changed NetworkRequest.url to proper getter/setter

This patch moves NetworkRequest.prototype.url to a proper getter/setter
function to be within our styleguide.

R=caseq,luoe,einbinder,dgozman
BUG=679105

Review-Url: https://codereview.chromium.org/2626553002
Cr-Commit-Position: refs/heads/master@{#442736}
Committed:
https://chromium.googlesource.com/chromium/src/+/8a062852968b7a329e313ab789bc...
==========

[commit-bot: I haz the power, 2017-01-10 23:58:46]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/8a062852968b7a329e313ab789bc...