Add more browser tests for Kiosk update from usb stick.

Add more browser tests for Kiosk update from usb stick.


Review URL: https://codereview.chromium.org/509253003


BUG=410611
Committed: https://crrev.com/9b2f59dc4294ea90ae3840ae1ab281eb6229b688
Cr-Commit-Position: refs/heads/master@{#293567}

[jennyz, 2014-09-04 00:10:06]

jennyz@chromium.org changed reviewers:
+ xiyuan@chromium.org

[xiyuan, 2014-09-04 03:57:02]

Can we get rid of KioskExternalUpdater::TestObserver and wire the update
complete event as part os KioskAppManagerObserver?

e.g.
  void OnExternalUpdateCompleted(bool success);

[jennyz, 2014-09-04 19:35:29]

On 2014/09/04 03:57:02, xiyuan wrote:
> Can we get rid of KioskExternalUpdater::TestObserver and wire the update
> complete event as part os KioskAppManagerObserver?
> 
> e.g.
>   void OnExternalUpdateCompleted(bool success);

Done, Please take another look. Thanks!

[xiyuan, 2014-09-04 19:49:54]

LGTM

Thank you for adding more tests!

[jennyz, 2014-09-04 20:35:40]

The CQ bit was checked by jennyz@chromium.org

[commit-bot: I haz the power, 2014-09-04 20:46:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jennyz@chromium.org/540673003/20001

[jennyz, 2014-09-04 22:31:20]

The CQ bit was unchecked by jennyz@chromium.org

[jennyz, 2014-09-04 22:31:22]

The CQ bit was checked by jennyz@chromium.org

[jennyz, 2014-09-04 22:32:06]

The CQ bit was unchecked by jennyz@chromium.org

[jennyz, 2014-09-04 23:04:24]

The CQ bit was checked by jennyz@chromium.org

[commit-bot: I haz the power, 2014-09-04 23:06:04]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jennyz@chromium.org/540673003/20001

[commit-bot: I haz the power, 2014-09-05 02:47:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-05 02:47:14]

Try jobs failed on following builders:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jennyz, 2014-09-05 16:34:06]

The CQ bit was checked by jennyz@chromium.org

[commit-bot: I haz the power, 2014-09-05 16:35:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jennyz@chromium.org/540673003/20001

[commit-bot: I haz the power, 2014-09-05 19:32:56]

Committed patchset #2 (id:20001) as b6941950130c3f063839ba6118717cc0d5251c05

[commit-bot: I haz the power, 2014-09-10 03:41:08]

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