While loading a page in tabmanager mode, control should come on content page hiding the tabmanager …

While loading a page in tabmanager mode, control should come on content page hiding the tabmanager mode.

Currently in TabManager window, if we try to load a url it gets loaded
in background.These changes transfer the control to content
from tab manager window, if a url is getting loaded.

Signed-off-by: divyabansal <divya.bansal@samsung.com>;

BUG=429977
Committed: https://crrev.com/d52161641eceb225c752261eb75f07556a2d24a4
Cr-Commit-Position: refs/heads/master@{#303223}

[divya.bansal, 2014-11-07 10:50:33]

divya.bansal@samsung.com changed reviewers:
+ bauerb@chromium.org, tedchoc@chromium.org

[divya.bansal, 2014-11-07 10:50:51]

PTAL

[Bernhard Bauer, 2014-11-07 11:02:39]

https://codereview.chromium.org/713443003/diff/1/chrome/android/shell/java/sr...
File
chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java
(right):

https://codereview.chromium.org/713443003/diff/1/chrome/android/shell/java/sr...
chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java:168:
if (mTabManager.isTabSwitcherVisible()) {
This should come before the comment, as the comment refers to lines 171-172.

[divya.bansal, 2014-11-07 11:38:34]

PTAL

https://codereview.chromium.org/713443003/diff/1/chrome/android/shell/java/sr...
File
chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java
(right):

https://codereview.chromium.org/713443003/diff/1/chrome/android/shell/java/sr...
chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java:168:
if (mTabManager.isTabSwitcherVisible()) {
On 2014/11/07 11:02:39, Bernhard Bauer wrote:
> This should come before the comment, as the comment refers to lines 171-172.

Done.

[Bernhard Bauer, 2014-11-07 11:40:31]

lgtm

[divya.bansal, 2014-11-07 11:40:58]

The CQ bit was checked by divya.bansal@samsung.com

[commit-bot: I haz the power, 2014-11-07 11:42:03]

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

[divya.bansal, 2014-11-07 11:47:57]

On 2014/11/07 11:42:03, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/713443003/20001

@Bernhard Thanks

[commit-bot: I haz the power, 2014-11-07 12:28:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-07 12:29:16]

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