[Telemetry] Modify CreatePageSet to CreateStorySet

[Telemetry] Modify CreatePageSet to CreateStorySet

BUG=465594
Committed: https://crrev.com/8af27d5cf40ebe4590fd5cc65880c9ab8b632ba4
Cr-Commit-Position: refs/heads/master@{#336713}

[nednguyen, 2015-06-29 19:19:36]

The CQ bit was checked by nednguyen@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-06-29 19:20:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217813005/1

[nednguyen, 2015-06-29 19:20:58]

nednguyen@google.com changed reviewers:
+ aiolos@chromium.org, eakuefner@chromium.org

[eakuefner, 2015-06-29 19:24:42]

lgtm

I'm wondering whether we want to care about references to "page set" in
comments/variable names (examples below). I'm also fine to punt this until we
address the TODO you added.

https://codereview.chromium.org/1217813005/diff/1/tools/telemetry/telemetry/i...
File tools/telemetry/telemetry/internal/util/find_dependencies.py (right):

https://codereview.chromium.org/1217813005/diff/1/tools/telemetry/telemetry/i...
tools/telemetry/telemetry/internal/util/find_dependencies.py:82: page_set =
test_obj.CreateStorySet(options)
nit: story_set?

https://codereview.chromium.org/1217813005/diff/1/tools/telemetry/telemetry/r...
File tools/telemetry/telemetry/record_wpr.py (right):

https://codereview.chromium.org/1217813005/diff/1/tools/telemetry/telemetry/r...
tools/telemetry/telemetry/record_wpr.py:165: def _GetPageSet(self, base_dir,
target):
nit: _GetStorySet?

[nednguyen, 2015-06-29 20:14:39]

nednguyen@google.com changed reviewers:
+ kbr@chromium.org

[nednguyen, 2015-06-29 20:25:02]

https://codereview.chromium.org/1217813005/diff/1/tools/telemetry/telemetry/r...
File tools/telemetry/telemetry/record_wpr.py (right):

https://codereview.chromium.org/1217813005/diff/1/tools/telemetry/telemetry/r...
tools/telemetry/telemetry/record_wpr.py:165: def _GetPageSet(self, base_dir,
target):
On 2015/06/29 19:24:42, eakuefner wrote:
> nit: _GetStorySet?

Done.

[Ken Russell (switch to Gerrit), 2015-06-29 22:57:11]

content/test/gpu LGTM as long as it passes the trybots.

[nednguyen, 2015-06-29 23:06:49]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2015-06-29 23:06:50]

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

[commit-bot: I haz the power, 2015-06-29 23:08:18]

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

[nednguyen, 2015-06-30 01:51:08]

The CQ bit was unchecked by nednguyen@google.com

[nednguyen, 2015-06-30 01:51:08]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2015-06-30 01:51:33]

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

[commit-bot: I haz the power, 2015-06-30 01:58:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-30 01:59:38]

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

[Rick Byers, 2015-06-30 18:45:12]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1217653003/ by rbyers@chromium.org.

The reason for reverting is: Speculative revert to attempt to address extreme
long-running telemetry jobs.  See http://crbug.com/505867 for details..