In AndroidBrowserBackend.Start(), make sure the application is stoped when setting the relaxsslchec…

Set the relaxsslcheck flag before stopping the browser.

BUG=

Committed: https://crrev.com/3910fcabde0fb21898a0bfb28fdcec7c302c6321
Cr-Commit-Position: refs/heads/master@{#293872}

[wuhu, 2014-09-08 21:16:25]

wuhu@google.com changed reviewers:
+ nednguyen@google.com, tony@chromium.org

[wuhu, 2014-09-08 21:16:26]

Please review when you get a chance.  Thanks

[nednguyen, 2014-09-08 21:31:27]

The actual fix should be removing Start method of browser.
possible_browser.Create() method returns a _running_ instance of browser
instead. Then the step of relaxing ssl can be put in __init__ and before the
step of killing running browser instance.

[tony, 2014-09-08 21:34:39]

tony@chromium.org changed reviewers:
+ tonyg@chromium.org
- tony@chromium.org

[tony, 2014-09-08 21:34:49]

(wrong tony)

[tonyg, 2014-09-08 22:27:55]

https://codereview.chromium.org/550403002/diff/1/tools/telemetry/telemetry/co...
File tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py
(right):

https://codereview.chromium.org/550403002/diff/1/tools/telemetry/telemetry/co...
tools/telemetry/telemetry/core/backends/chrome/android_browser_backend.py:201:
self._adb.device().ForceStop(self._backend_settings.package)
We already do this here. Can we just make sure your code runs before it?

[wuhu, 2014-09-08 23:36:12]

Moved relaxsslcheck to the constructor.  Thanks.

[tonyg, 2014-09-08 23:40:22]

The CQ bit was checked by tonyg@chromium.org

[tonyg, 2014-09-08 23:40:22]

lgtm

[commit-bot: I haz the power, 2014-09-08 23:43:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wuhu@google.com/550403002/20001

[nednguyen, 2014-09-08 23:44:23]

lgtm since tony is happy

[commit-bot: I haz the power, 2014-09-09 06:30:19]

Committed patchset #2 (id:20001) as f4e8aedd868d18f7f0fd8dec6449bd737b08c864

[commit-bot: I haz the power, 2014-09-10 03:51:16]

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