[system health] Fix wpr archive names.

[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
Committed: https://crrev.com/d7cfe1197875c332ba6116a0bd2d6c08c2235d84
Cr-Commit-Position: refs/heads/master@{#405998}

[ulan, 2016-07-18 13:46:47]

ulan@chromium.org changed reviewers:
+ nednguyen@google.com, petrcermak@chromium.org

[ulan, 2016-07-18 13:46:47]

PTAL.

I didn't upload the wpr files to cloud storage yet, but presubmit check did not
complain.

I will upload after getting your approval.

[ulan, 2016-07-18 13:49:11]

Description was changed from

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.
==========

to

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21
==========

[petrcermak, 2016-07-18 13:49:39]

The stories actually have nothing to do with memory. Could you please rename
them to "system_health_desktop.json" and "system_health_mobile.json"?

Apart from that, LGTM. Thanks a lot for taking care of this and sorry for
causing the trouble in the first place.

Petr

[nednguyen, 2016-07-18 13:49:43]

On 2016/07/18 13:46:47, ulan wrote:
> PTAL.
> 
> I didn't upload the wpr files to cloud storage yet, but presubmit check did
not
> complain.
> 
> I will upload after getting your approval.

lgtm Can you also delete the old *.wpr.sha1 files?

[petrcermak, 2016-07-18 13:50:15]

Also please add "BUG=589726".

[ulan, 2016-07-18 13:51:38]

Description was changed from

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21
==========

to

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
==========

[ulan, 2016-07-18 14:05:25]

> The stories actually have nothing to do with memory. Could you please rename
> them to "system_health_desktop.json" and "system_health_mobile.json"?
Done.

> Apart from that, LGTM. Thanks a lot for taking care of this and sorry for
> causing the trouble in the first place.
No problem :)

> lgtm Can you also delete the old *.wpr.sha1 files?
Done.

> Also please add "BUG=589726".
Done.

I am going to upload wpr files now.

[ulan, 2016-07-18 14:24:56]

> I am going to upload wpr files now.
Actually, there is no need to upload wpr files: the sha hash is used as the key
in google storage. Since we didn't change the hash and the contents of wpr,
google storage should contain correct data.

I checked it in a separate chrome checkout and it worked. I will land after
running try bots.

[ulan, 2016-07-18 14:26:36]

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

[commit-bot: I haz the power, 2016-07-18 14:26:56]

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

[ulan, 2016-07-18 14:30:36]

Description was changed from

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
==========

to

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
CQ_INCLUDE_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq
==========

[ulan, 2016-07-18 14:31:03]

The CQ bit was unchecked by ulan@chromium.org

[ulan, 2016-07-18 14:32:25]

The CQ bit was checked by ulan@chromium.org

[ulan, 2016-07-18 14:32:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
nednguyen@google.com, petrcermak@chromium.org
Link to the patchset: https://codereview.chromium.org/2161583003/#ps40001
(title: "add json files")

[commit-bot: I haz the power, 2016-07-18 14:32:54]

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-07-18 14:32:56]

Your CL relies on deprecated CQ feature(s):
 * Specifying master names in CQ_INCLUDE_TRYBOTS part of description without
"master." prefix is deprecated:
     tryserver.chromium.perf
   For more details, see http://crbug.com/617627.

[nednguyen, 2016-07-18 15:01:47]

Description was changed from

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
CQ_INCLUDE_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq
==========

to

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
==========

[commit-bot: I haz the power, 2016-07-18 15:34:47]

Description was changed from

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
==========

to

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
==========

[commit-bot: I haz the power, 2016-07-18 15:34:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-18 15:41:13]

Description was changed from

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726
==========

to

==========
[system health] Fix wpr archive names.

This patch renames browsing_*.wpr to memory_system_health_*.wpr so
that all references in memory_system_health_*.json start with the same
prefix 'memory_system_health'.

For more context see https://codereview.chromium.org/2144073004/#msg21

BUG=589726

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

[commit-bot: I haz the power, 2016-07-18 15:41:14]

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