[tools/perf] Move system_health_stories to system_health.loading_stories module

[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single
system_health story set, hence the current direction is to create multiple
stories set for key user actions. In the future, the
page_sets/system_health_stories/ folder can contain these extra
story_sets:
loading
scrolling & animation
multi_tabs
...

BUG=615382, 589726
Committed: https://crrev.com/0087d3d5b48bcda9d7173769369c344e33f205c7
Cr-Commit-Position: refs/heads/master@{#397692}

[nednguyen, 2016-06-03 01:51:35]

Description was changed from

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module
==========

to

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single 
system_health story set, hence the current direction is to create multiple 
stories set for key user actions. In the future, the 
page_sets/system_health_stories/ folder can contain these extra 
story_sets:
loading
scrolling & animation 
multi_tabs
...
==========

[nednguyen, 2016-06-03 01:52:52]

nednguyen@google.com changed reviewers:
+ benjhayden@chromium.org, charliea@chromium.org, kouhei@chromium.org,
petrcermak@chromium.org

[nednguyen, 2016-06-03 01:53:08]

On 2016/06/03 01:52:52, nednguyen wrote:

PTAL

[kouhei (in TOK), 2016-06-03 02:00:17]

lgtm

[petrcermak, 2016-06-03 09:04:09]

I'm fine with this, but I think you need to update the system health benchmark
(https://cs.chromium.org/chromium/src/tools/perf/benchmarks/system_health.py?l=80
and
https://cs.chromium.org/chromium/src/tools/perf/benchmarks/system_health.py?l=90)
accordingly.

Thanks,
Petr

[petrcermak, 2016-06-03 09:04:09]

I'm fine with this, but I think you need to update the system health benchmark
(https://cs.chromium.org/chromium/src/tools/perf/benchmarks/system_health.py?l=80
and
https://cs.chromium.org/chromium/src/tools/perf/benchmarks/system_health.py?l=90)
accordingly.

Thanks,
Petr

[nednguyen, 2016-06-03 11:39:39]

On 2016/06/03 09:04:09, petrcermak wrote:
> I'm fine with this, but I think you need to update the system health benchmark
>
(https://cs.chromium.org/chromium/src/tools/perf/benchmarks/system_health.py?l=80
> and
>
https://cs.chromium.org/chromium/src/tools/perf/benchmarks/system_health.py?l=90)
> accordingly.
> 
> Thanks,
> Petr

That doesn't need to be updated due to the magic in
https://cs.chromium.org/chromium/src/tools/perf/page_sets/__init__.py?rcl=0&l=13

[petrcermak, 2016-06-03 11:53:28]

Cool! LGTM.

Thanks,
Petr

[nednguyen, 2016-06-03 12:55:56]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2016-06-03 12:55:57]

The patchset sent to the CQ was uploaded after l-g-t-m from kouhei@chromium.org
Link to the patchset: https://codereview.chromium.org/2033693005/#ps20001
(title: "Add __init__")

[nednguyen, 2016-06-03 12:56:02]

The CQ bit was unchecked by nednguyen@google.com

[commit-bot: I haz the power, 2016-06-03 12:56:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2033693005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2033693005/20001

[nednguyen, 2016-06-03 12:56:58]

Description was changed from

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single 
system_health story set, hence the current direction is to create multiple 
stories set for key user actions. In the future, the 
page_sets/system_health_stories/ folder can contain these extra 
story_sets:
loading
scrolling & animation 
multi_tabs
...
==========

to

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single 
system_health story set, hence the current direction is to create multiple 
stories set for key user actions. In the future, the 
page_sets/system_health_stories/ folder can contain these extra 
story_sets:
loading
scrolling & animation 
multi_tabs
...

BUG=615382, 589726
==========

[commit-bot: I haz the power, 2016-06-03 12:59:44]

Description was changed from

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single 
system_health story set, hence the current direction is to create multiple 
stories set for key user actions. In the future, the 
page_sets/system_health_stories/ folder can contain these extra 
story_sets:
loading
scrolling & animation 
multi_tabs
...

BUG=615382, 589726
==========

to

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single 
system_health story set, hence the current direction is to create multiple 
stories set for key user actions. In the future, the 
page_sets/system_health_stories/ folder can contain these extra 
story_sets:
loading
scrolling & animation 
multi_tabs
...

BUG=615382, 589726
==========

[commit-bot: I haz the power, 2016-06-03 12:59:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-03 13:02:26]

Description was changed from

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single 
system_health story set, hence the current direction is to create multiple 
stories set for key user actions. In the future, the 
page_sets/system_health_stories/ folder can contain these extra 
story_sets:
loading
scrolling & animation 
multi_tabs
...

BUG=615382, 589726
==========

to

==========
[tools/perf] Move system_health_stories to system_health.loading_stories module

Rationale: we know that it's almost impossible to have 1 single
system_health story set, hence the current direction is to create multiple
stories set for key user actions. In the future, the
page_sets/system_health_stories/ folder can contain these extra
story_sets:
loading
scrolling & animation
multi_tabs
...

BUG=615382, 589726

Committed: https://crrev.com/0087d3d5b48bcda9d7173769369c344e33f205c7
Cr-Commit-Position: refs/heads/master@{#397692}
==========

[commit-bot: I haz the power, 2016-06-03 13:02:26]

Patchset 2 (id:??) landed as
https://crrev.com/0087d3d5b48bcda9d7173769369c344e33f205c7
Cr-Commit-Position: refs/heads/master@{#397692}