Enabling TabManager on testing browser process

To make it possible run all TabManager based browsertests as unittest.

BUG=625850
Committed: https://crrev.com/13af30c7bfb22dae8f79ebe203d959c71c711a37
Cr-Commit-Position: refs/heads/master@{#403960}

[Anderson Silva, 2016-07-06 16:58:03]

Description was changed from

==========
fixing spaces


enabling TabManager on testing browser process

BUG=625850
==========

to

==========
To make it possible run all TabManager based browsertests as unittest.

BUG=625850
==========

[Anderson Silva, 2016-07-06 16:58:04]

andersoncss@google.com changed reviewers:
+ chrisha@chromium.org, georgesak@chromium.org

[Anderson Silva, 2016-07-06 17:54:29]

PTAL

[Georges Khalil, 2016-07-06 18:00:04]

LGTM % nit.

https://codereview.chromium.org/2121403002/diff/20001/chrome/test/base/testin...
File chrome/test/base/testing_browser_process.h (right):

https://codereview.chromium.org/2121403002/diff/20001/chrome/test/base/testin...
chrome/test/base/testing_browser_process.h:183: // invoked in suppoerted
platforms.
nit (wording): TabManager is null by default and will be created when
GetTabManager() is invoked on supported platforms.

[Georges Khalil, 2016-07-06 18:00:45]

Modified comment.

https://codereview.chromium.org/2121403002/diff/20001/chrome/test/base/testin...
File chrome/test/base/testing_browser_process.h (right):

https://codereview.chromium.org/2121403002/diff/20001/chrome/test/base/testin...
chrome/test/base/testing_browser_process.h:183: // invoked in suppoerted
platforms.
On 2016/07/06 18:00:04, Georges Khalil wrote:
> nit (wording): TabManager is null by default and will be created when
> GetTabManager() is invoked on supported platforms.

Make that: |tab_manager_| is null...

[Anderson Silva, 2016-07-06 18:11:11]

Changed comment.

https://codereview.chromium.org/2121403002/diff/20001/chrome/test/base/testin...
File chrome/test/base/testing_browser_process.h (right):

https://codereview.chromium.org/2121403002/diff/20001/chrome/test/base/testin...
chrome/test/base/testing_browser_process.h:183: // invoked in suppoerted
platforms.
On 2016/07/06 18:00:04, Georges Khalil wrote:
> nit (wording): TabManager is null by default and will be created when
> GetTabManager() is invoked on supported platforms.

Done.

[Anderson Silva, 2016-07-06 18:43:20]

andersoncss@google.com changed reviewers:
+ sky@chromium.org

[Anderson Silva, 2016-07-06 18:43:21]

Hi, 
Can you please take a look at this?

Thanks

[sky, 2016-07-06 20:34:08]

LGTM

[Anderson Silva, 2016-07-06 20:56:17]

The CQ bit was checked by andersoncss@google.com

[Anderson Silva, 2016-07-06 20:56:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
georgesak@chromium.org
Link to the patchset: https://codereview.chromium.org/2121403002/#ps60001
(title: "modified comment")

[commit-bot: I haz the power, 2016-07-06 20:57:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-06 22:09:17]

Description was changed from

==========
To make it possible run all TabManager based browsertests as unittest.

BUG=625850
==========

to

==========
To make it possible run all TabManager based browsertests as unittest.

BUG=625850
==========

[commit-bot: I haz the power, 2016-07-06 22:09:18]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-06 22:09:44]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-06 22:11:07]

Description was changed from

==========
To make it possible run all TabManager based browsertests as unittest.

BUG=625850
==========

to

==========
To make it possible run all TabManager based browsertests as unittest.

BUG=625850

Committed: https://crrev.com/13af30c7bfb22dae8f79ebe203d959c71c711a37
Cr-Commit-Position: refs/heads/master@{#403960}
==========

[commit-bot: I haz the power, 2016-07-06 22:11:08]

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