Remove TabModelSelector's dependency on ChromeFullscreenManager.

Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939
Committed: https://crrev.com/820a95b0b81d33e42712f9198c215f703412e1a1
Cr-Commit-Position: refs/heads/master@{#426935}

[Ted C, 2016-10-20 00:03:51]

Description was changed from

==========
Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939

Add an explicit initializeTabModelSelector call to ChromeActivity.

ChromeActivity's initialization path has gotten a bit unwieldly
and this attempts to bring some sanity to that.

BUG=
==========

to

==========
Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939
==========

[Ted C, 2016-10-20 00:04:43]

tedchoc@chromium.org changed reviewers:
+ yusufo@chromium.org

[Ted C, 2016-10-20 00:04:43]

Here is the change that fixes the ChromeFullscreenManager dependency and should
allow CCT to initialize the tab models early as desired.

[Ted C, 2016-10-20 22:46:49]

The dependent change landed and I rebased this, so this should be good to be
reviewed.

[Yusuf, 2016-10-21 20:03:36]

lgtm

[Ted C, 2016-10-21 20:22:50]

The CQ bit was checked by tedchoc@chromium.org

[Ted C, 2016-10-21 20:22:50]

The patchset sent to the CQ was uploaded after l-g-t-m from yusufo@chromium.org
Link to the patchset: https://codereview.chromium.org/2440643002/#ps40001
(title: "Rebased")

[commit-bot: I haz the power, 2016-10-21 20:23:33]

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-10-21 22:01:11]

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

[commit-bot: I haz the power, 2016-10-21 22:01:11]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Ted C, 2016-10-21 22:12:22]

The CQ bit was checked by tedchoc@chromium.org

[Ted C, 2016-10-21 22:12:22]

The patchset sent to the CQ was uploaded after l-g-t-m from yusufo@chromium.org
Link to the patchset: https://codereview.chromium.org/2440643002/#ps60001
(title: "file move")

[commit-bot: I haz the power, 2016-10-21 22:13:13]

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-10-22 00:29:53]

Description was changed from

==========
Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939
==========

to

==========
Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939
==========

[commit-bot: I haz the power, 2016-10-22 00:29:54]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-22 00:32:45]

Description was changed from

==========
Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939
==========

to

==========
Remove TabModelSelector's dependency on ChromeFullscreenManager.

ChromeFullscreenManager will now register and listen
for updates internally and avoids the prior cyclical
dependency.

This also fixes a bug introduced in:
https://codereview.chromium.org/2414913004

In that, we were not clearing the reference to Tab when
reparenting and then there were two fullscreen managers
listening for updates.

BUG=656939

Committed: https://crrev.com/820a95b0b81d33e42712f9198c215f703412e1a1
Cr-Commit-Position: refs/heads/master@{#426935}
==========

[commit-bot: I haz the power, 2016-10-22 00:32:46]

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