Cosmetic changes to chrome://net-export, primarily for desktop mode.

Cosmetic changes to chrome://net-export, primarily for desktop mode.

 * On the desktop UI, remove reference to emailing from the instructions
   (as that is only available from the mobile version).

 * While logging is in progress, change the disabled button's text from:
     "Start Logging to Disk"  to "Logging to Disk..."
   This gives a bit more feedback that something is happening (even
   better would be to show the path to log file and its size)

 * Add a <title> for the page of "Network Log Export"

BUG=701033, 701993
Review-Url: https://codereview.chromium.org/2757473002
Cr-Commit-Position: refs/heads/master@{#457612}
Committed: https://chromium.googlesource.com/chromium/src/+/62a3acd978daee61b3ea99b4cd705d4401203ee3

[eroman, 2017-03-16 00:21:29]

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

[commit-bot: I haz the power, 2017-03-16 00:22:04]

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

[eroman, 2017-03-16 00:28:20]

eroman@chromium.org changed reviewers:
+ xunjieli@chromium.org

[eroman, 2017-03-16 00:41:35]

Description was changed from

==========
Cosmetic changes to chrome://net-export, primarily for desktop mode.

 * On the desktop UI, remove reference to emailing from the instructions
   (as that is only vailable from the mobile version).

 * While logging is in progress, change the disabled button's text from:
     "Start Logging to Disk"  to "Logging to Disk..."
   This gives a bit more feedback that something is happening (even
   better would be to show the path to log file and its size)

 * Add a <title> for the page of "Network Log Export"

BUG=701033,701993
==========

to

==========
Cosmetic changes to chrome://net-export, primarily for desktop mode.

 * On the desktop UI, remove reference to emailing from the instructions
   (as that is only available from the mobile version).

 * While logging is in progress, change the disabled button's text from:
     "Start Logging to Disk"  to "Logging to Disk..."
   This gives a bit more feedback that something is happening (even
   better would be to show the path to log file and its size)

 * Add a <title> for the page of "Network Log Export"

BUG=701033,701993
==========

[commit-bot: I haz the power, 2017-03-16 01:33:16]

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

[commit-bot: I haz the power, 2017-03-16 01:33:17]

Dry run: This issue passed the CQ dry run.

[xunjieli, 2017-03-16 19:49:49]

I tried the change on desktop and on Android. 
I can see why the "Advanced" option to include all bytes might be slightly
confusing on Mobile since the user might not know changing the option doesn't do
anything for an existing log file. I guess that could be handled later. This CL
lgtm.

https://codereview.chromium.org/2757473002/diff/1/components/net_log/resource...
File components/net_log/resources/net_export.html (right):

https://codereview.chromium.org/2757473002/diff/1/components/net_log/resource...
components/net_log/resources/net_export.html:67: Logs files can be loaded using
nit: maybe "Logs files" -> "Log files"

[eroman, 2017-03-16 21:46:56]

Thanks for the review!

Agreed the capture modes are still confusing. I think we have a good strategy
for resolving this described at:

https://bugs.chromium.org/p/chromium/issues/detail?id=701033#c8

But I will not address that in this CL.

https://codereview.chromium.org/2757473002/diff/1/components/net_log/resource...
File components/net_log/resources/net_export.html (right):

https://codereview.chromium.org/2757473002/diff/1/components/net_log/resource...
components/net_log/resources/net_export.html:67: Logs files can be loaded using
On 2017/03/16 19:49:49, xunjieli wrote:
> nit: maybe "Logs files" -> "Log files"

Done.

[eroman, 2017-03-16 21:48:30]

The CQ bit was checked by eroman@chromium.org

[eroman, 2017-03-16 21:48:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
xunjieli@chromium.org
Link to the patchset: https://codereview.chromium.org/2757473002/#ps20001
(title: "address Helen's comments")

[commit-bot: I haz the power, 2017-03-16 21:49: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-03-16 23:52:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489700910774550,
"parent_rev": "f0e9ed11cfd03e38d41a5abad3de3c4501ddbcae", "commit_rev":
"62a3acd978daee61b3ea99b4cd705d4401203ee3"}

[commit-bot: I haz the power, 2017-03-16 23:53:38]

Description was changed from

==========
Cosmetic changes to chrome://net-export, primarily for desktop mode.

 * On the desktop UI, remove reference to emailing from the instructions
   (as that is only available from the mobile version).

 * While logging is in progress, change the disabled button's text from:
     "Start Logging to Disk"  to "Logging to Disk..."
   This gives a bit more feedback that something is happening (even
   better would be to show the path to log file and its size)

 * Add a <title> for the page of "Network Log Export"

BUG=701033,701993
==========

to

==========
Cosmetic changes to chrome://net-export, primarily for desktop mode.

 * On the desktop UI, remove reference to emailing from the instructions
   (as that is only available from the mobile version).

 * While logging is in progress, change the disabled button's text from:
     "Start Logging to Disk"  to "Logging to Disk..."
   This gives a bit more feedback that something is happening (even
   better would be to show the path to log file and its size)

 * Add a <title> for the page of "Network Log Export"

BUG=701033,701993

Review-Url: https://codereview.chromium.org/2757473002
Cr-Commit-Position: refs/heads/master@{#457612}
Committed:
https://chromium.googlesource.com/chromium/src/+/62a3acd978daee61b3ea99b4cd70...
==========

[commit-bot: I haz the power, 2017-03-16 23:53:39]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/62a3acd978daee61b3ea99b4cd70...