Fix fullscreen browser overrides when no tab is present.

Fix fullscreen browser overrides when no tab is present.

If a client requests the top controls be made visible via
the BrowserStateBrowserControlsVisibilityDelegate but no
tabs are available, the fullscreen manager should force
the controls to be visible anyway.

This fixes an issue where you only have a single incognito
tab, scroll off the top controls, then hit the close all
incognito tabs notification.  At this point, you are jumped
back to the tab switcher but the controls were not being
shown without this additional logic.

BUG=675518
Review-Url: https://codereview.chromium.org/2647943002
Cr-Commit-Position: refs/heads/master@{#445157}
Committed: https://chromium.googlesource.com/chromium/src/+/c5be519e4e6afeed39e9b8f94dc46731cd844997

[Ted C, 2017-01-20 18:52:37]

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

[Ted C, 2017-01-20 18:52:37]

PTAL

[mdjones, 2017-01-20 19:56:51]

lgtm

[Ted C, 2017-01-20 20:13:49]

The CQ bit was checked by tedchoc@chromium.org

[commit-bot: I haz the power, 2017-01-20 20:14:12]

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, 2017-01-20 20:54:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484943229378330, "parent_rev":
"eecead3bf528702cb98ab2c09885055b26ca8f46", "commit_rev":
"c5be519e4e6afeed39e9b8f94dc46731cd844997"}

[commit-bot: I haz the power, 2017-01-20 20:54:51]

Description was changed from

==========
Fix fullscreen browser overrides when no tab is present.

If a client requests the top controls be made visible via
the BrowserStateBrowserControlsVisibilityDelegate but no
tabs are available, the fullscreen manager should force
the controls to be visible anyway.

This fixes an issue where you only have a single incognito
tab, scroll off the top controls, then hit the close all
incognito tabs notification.  At this point, you are jumped
back to the tab switcher but the controls were not being
shown without this additional logic.

BUG=675518
==========

to

==========
Fix fullscreen browser overrides when no tab is present.

If a client requests the top controls be made visible via
the BrowserStateBrowserControlsVisibilityDelegate but no
tabs are available, the fullscreen manager should force
the controls to be visible anyway.

This fixes an issue where you only have a single incognito
tab, scroll off the top controls, then hit the close all
incognito tabs notification.  At this point, you are jumped
back to the tab switcher but the controls were not being
shown without this additional logic.

BUG=675518

Review-Url: https://codereview.chromium.org/2647943002
Cr-Commit-Position: refs/heads/master@{#445157}
Committed:
https://chromium.googlesource.com/chromium/src/+/c5be519e4e6afeed39e9b8f94dc4...
==========

[commit-bot: I haz the power, 2017-01-20 20:54:52]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c5be519e4e6afeed39e9b8f94dc4...