[Mac] Do not initialize the MockCrApplication in base::TestSuite.

[Mac] Do not initialize the MockCrApplication in base::TestSuite.

This is not the right layer to do this, since it forces all test suites to have
an initialized Cocoa NSApp. Instead, only do it for test suites that require it.

BUG=71686
Committed: https://crrev.com/6a398155ce27db6153ca1c39eb6380d624fc8e01
Cr-Commit-Position: refs/heads/master@{#300275}

[Robert Sesek, 2014-10-17 18:40:05]

rsesek@chromium.org changed reviewers:
+ mark@chromium.org

[Mark Mentovai, 2014-10-17 19:44:58]

LGTM

https://codereview.chromium.org/656293003/diff/40001/base/mac/scoped_sending_...
File base/mac/scoped_sending_event_unittest.mm (right):

https://codereview.chromium.org/656293003/diff/40001/base/mac/scoped_sending_...
base/mac/scoped_sending_event_unittest.mm:11: 
:)

https://codereview.chromium.org/656293003/diff/40001/ui/app_list/test/run_all...
File ui/app_list/test/run_all_unittests.cc (right):

https://codereview.chromium.org/656293003/diff/40001/ui/app_list/test/run_all...
ui/app_list/test/run_all_unittests.cc:15: #if !defined(OS_MACOSX)
Now that these have an else clause, they’d probably be a little more
straightforward to read if the “if” weren’t negated.

[Robert Sesek, 2014-10-17 21:02:29]

rsesek@chromium.org changed reviewers:
+ sky@chromium.org

[Robert Sesek, 2014-10-17 21:02:29]

+sky for //ui OWNERS

https://codereview.chromium.org/656293003/diff/40001/ui/app_list/test/run_all...
File ui/app_list/test/run_all_unittests.cc (right):

https://codereview.chromium.org/656293003/diff/40001/ui/app_list/test/run_all...
ui/app_list/test/run_all_unittests.cc:15: #if !defined(OS_MACOSX)
On 2014/10/17 19:44:58, Mark Mentovai wrote:
> Now that these have an else clause, they’d probably be a little more
> straightforward to read if the “if” weren’t negated.

Done.

[Robert Sesek, 2014-10-20 16:16:07]

sky: ping

[sky, 2014-10-20 16:16:28]

LGTM

[Robert Sesek, 2014-10-20 16:17:02]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2014-10-20 16:17:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/656293003/60001

[commit-bot: I haz the power, 2014-10-20 16:35:25]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-10-20 16:36:36]

Patchset 4 (id:??) landed as
https://crrev.com/6a398155ce27db6153ca1c39eb6380d624fc8e01
Cr-Commit-Position: refs/heads/master@{#300275}