[Devtools] Moved preserve log to a setting

[Devtools] Moved preserve log to a setting

Moves preserve log to a setting. This should allow it to be changed in
the actions menu and will allow NetworkLog to check the state of
preserve log for followup patch to move some functionality into
NetworkLog.

R=dgozman,pfeldman
BUG=None

Review-Url: https://codereview.chromium.org/2903183002
Cr-Commit-Position: refs/heads/master@{#474887}
Committed: https://chromium.googlesource.com/chromium/src/+/c4c1cb3507777d5bbef9bc09c87571fe4a768e41

[allada, 2017-05-25 00:57:11]

Patchset #1 (id:1) has been deleted

[allada, 2017-05-25 01:00:44]

Patchset #1 (id:20001) has been deleted

[allada, 2017-05-25 01:02:19]

Patchset #1 (id:40001) has been deleted

[allada, 2017-05-25 01:03:01]

PTL

[dgozman, 2017-05-25 23:07:36]

lgtm

https://codereview.chromium.org/2903183002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/network/module.json (right):

https://codereview.chromium.org/2903183002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/network/module.json:67: "category":
"Network",
Make it session setting to preserve current behavior.

https://codereview.chromium.org/2903183002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/network/module.json:72: "tags":
"network, preserve, clear, reset, log",
No need to repeat words from description (preserve and log in this case).

[allada, 2017-05-26 00:03:51]

The CQ bit was checked by allada@chromium.org

[allada, 2017-05-26 00:03:52]

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/2903183002/#ps80001
(title: "changes")

[allada, 2017-05-26 00:04:00]

done.

https://codereview.chromium.org/2903183002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/network/module.json (right):

https://codereview.chromium.org/2903183002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/network/module.json:67: "category":
"Network",
On 2017/05/25 23:07:36, dgozman wrote:
> Make it session setting to preserve current behavior.

Done.

https://codereview.chromium.org/2903183002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/network/module.json:72: "tags":
"network, preserve, clear, reset, log",
On 2017/05/25 23:07:36, dgozman wrote:
> No need to repeat words from description (preserve and log in this case).

Done.

[commit-bot: I haz the power, 2017-05-26 00:04: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, 2017-05-26 02:44:30]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1495757031723990,
"parent_rev": "2e2b178d99399c687bc60adea7e6b2460c9cfca6", "commit_rev":
"c4c1cb3507777d5bbef9bc09c87571fe4a768e41"}

[commit-bot: I haz the power, 2017-05-26 02:44:44]

Description was changed from

==========
[Devtools] Moved preserve log to a setting

Moves preserve log to a setting. This should allow it to be changed in
the actions menu and will allow NetworkLog to check the state of
preserve log for followup patch to move some functionality into
NetworkLog.

R=dgozman,pfeldman
BUG=None
==========

to

==========
[Devtools] Moved preserve log to a setting

Moves preserve log to a setting. This should allow it to be changed in
the actions menu and will allow NetworkLog to check the state of
preserve log for followup patch to move some functionality into
NetworkLog.

R=dgozman,pfeldman
BUG=None

Review-Url: https://codereview.chromium.org/2903183002
Cr-Commit-Position: refs/heads/master@{#474887}
Committed:
https://chromium.googlesource.com/chromium/src/+/c4c1cb3507777d5bbef9bc09c875...
==========

[commit-bot: I haz the power, 2017-05-26 02:44:47]

Committed patchset #2 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/c4c1cb3507777d5bbef9bc09c875...