🐻 Fix ui_base_unittests on Android Swarming (adds missing data)

Fix ui_base_unittests on Android Swarming (adds missing data)

BUG=589318
Committed: https://crrev.com/f9917d1e7bc35236ad1e49f888f608c11d563ecf
Cr-Commit-Position: refs/heads/master@{#389845}

[agrieve, 2016-04-26 01:39:38]

agrieve@chromium.org changed reviewers:
+ jbudorick@chromium.org, sadrul@chromium.org

[jbudorick, 2016-04-26 15:51:49]

lgtm w/ nits

https://codereview.chromium.org/1922023002/diff/1/build/android/pylib/utils/i...
File build/android/pylib/utils/isolator.py (right):

https://codereview.chromium.org/1922023002/diff/1/build/android/pylib/utils/i...
build/android/pylib/utils/isolator.py:155: @staticmethod
nit: @classmethod

https://codereview.chromium.org/1922023002/diff/1/build/android/pylib/utils/i...
build/android/pylib/utils/isolator.py:159:
Isolator._DestructiveMerge(os.path.join(src, p), os.path.join(dest, p))
nit: cls._DestructiveMerge

[sadrul, 2016-04-26 16:26:51]

stamp lgtm

[agrieve, 2016-04-26 17:00:34]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2016-04-26 17:00:37]

The CQ bit was unchecked by agrieve@chromium.org

[agrieve, 2016-04-26 17:34:33]

https://codereview.chromium.org/1922023002/diff/1/build/android/pylib/utils/i...
File build/android/pylib/utils/isolator.py (right):

https://codereview.chromium.org/1922023002/diff/1/build/android/pylib/utils/i...
build/android/pylib/utils/isolator.py:155: @staticmethod
On 2016/04/26 15:51:49, jbudorick wrote:
> nit: @classmethod

Done.

https://codereview.chromium.org/1922023002/diff/1/build/android/pylib/utils/i...
build/android/pylib/utils/isolator.py:159:
Isolator._DestructiveMerge(os.path.join(src, p), os.path.join(dest, p))
On 2016/04/26 15:51:49, jbudorick wrote:
> nit: cls._DestructiveMerge

Done.

[agrieve, 2016-04-26 17:34:41]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2016-04-26 17:34:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org, sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/1922023002/#ps20001
(title: "@classmethod")

[commit-bot: I haz the power, 2016-04-26 17:34:53]

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

[commit-bot: I haz the power, 2016-04-26 19:20:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-26 19:22:01]

Description was changed from

==========
Fix ui_base_unittests on Android Swarming (adds missing data)

BUG=589318
==========

to

==========
Fix ui_base_unittests on Android Swarming (adds missing data)

BUG=589318

Committed: https://crrev.com/f9917d1e7bc35236ad1e49f888f608c11d563ecf
Cr-Commit-Position: refs/heads/master@{#389845}
==========

[commit-bot: I haz the power, 2016-04-26 19:22:02]

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