Simplifies ShellTestHelper and ViewManager tests

Simplifies ShellTestHelper and ViewManager tests

ShellTestHelper doesn't need to spawn a thread. Nuking the thread
simplifies the view manager tests.

BUG=365012
TEST=covered by tests
R=ben@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274167

[Ben Goodger (Google), 2014-05-30 22:00:27]

lgtm

[sky, 2014-05-30 22:02:11]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-05-30 22:05:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/308053008/10001

[commit-bot: I haz the power, 2014-05-31 03:11:18]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[commit-bot: I haz the power, 2014-05-31 04:31:23]

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

[commit-bot: I haz the power, 2014-05-31 04:31:24]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[sky, 2014-06-01 19:48:52]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-06-01 19:49:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/308053008/30001

[commit-bot: I haz the power, 2014-06-02 05:34:23]

Change committed as 274167