[Telemetry] Change API of ShouldStopBrowserAfterStoryRun

[Telemetry] Change API of ShouldStopBrowserAfterStoryRun

Some clients that will override this shared state method need to know
what the current story is to decide whether the browser should be
closed or not.

BUG=chromium:748566
Review-Url: https://codereview.chromium.org/2994833002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/5b7f1a76070bcbac5af706cdf89b7bcdbb68f380

[perezju, 2017-08-10 13:40:07]

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

[perezju, 2017-08-10 13:40:08]

This small change needed to unblock src-side clients making the switch.

Nobody is using or overriding the method yet, so the change is safe.

[nednguyen, 2017-08-10 14:06:25]

lgtm

I guess these are ones with story.startup_url?

[perezju, 2017-08-10 14:20:30]

On 2017/08/10 14:06:25, nednguyen wrote:
> lgtm
> 
> I guess these are ones with story.startup_url?

No, this is required for memory.top_10_mobile in
https://chromium-review.googlesource.com/c/610044

This should not affect stories with startup_url,
which use the default behavior and always close
the browser after the story is done.

[perezju, 2017-08-10 14:24:49]

The CQ bit was checked by perezju@chromium.org

[commit-bot: I haz the power, 2017-08-10 14:24: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, 2017-08-10 15:18:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1502375089681550, "parent_rev":
"8a1ca7ff3ad0a5c4af93ba5b018c2c7d74cc276f", "commit_rev":
"5b7f1a76070bcbac5af706cdf89b7bcdbb68f380"}

[commit-bot: I haz the power, 2017-08-10 15:18:09]

Description was changed from

==========
[Telemetry] Change API of ShouldStopBrowserAfterStoryRun

Some clients that will override this shared state method need to know
what the current story is to decide whether the browser should be
closed or not.

BUG=chromium:748566
==========

to

==========
[Telemetry] Change API of ShouldStopBrowserAfterStoryRun

Some clients that will override this shared state method need to know
what the current story is to decide whether the browser should be
closed or not.

BUG=chromium:748566

Review-Url: https://codereview.chromium.org/2994833002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-08-10 15:18:10]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...