Commit 742a867b authored by Yining Wang's avatar Yining Wang Committed by Commit Bot

Fix Toolbar content for Focus Mode Window

Toolbar is not showing correct content for Focus Mode Window when the
site is not secure. This cl resolve this issue by processing an UI
update after tab inserted to browser.

Change-Id: I35c4baf20ba6c90c85a03f471c51356853e8f165

Bug: 961820
Change-Id: I35c4baf20ba6c90c85a03f471c51356853e8f165
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603768
Commit-Queue: Yining Wang <yiningwang@google.com>
Reviewed-by: default avatarDana Fried <dfried@chromium.org>
Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#658820}
parent b7f65b1f
...@@ -2093,6 +2093,7 @@ void Browser::OnTabInsertedAt(WebContents* contents, int index) { ...@@ -2093,6 +2093,7 @@ void Browser::OnTabInsertedAt(WebContents* contents, int index) {
if (index < new_active_index) if (index < new_active_index)
session_service->SetSelectedTabInWindow(session_id(), new_active_index); session_service->SetSelectedTabInWindow(session_id(), new_active_index);
} }
contents->NotifyNavigationStateChanged(content::INVALIDATE_TYPE_TAB);
} }
void Browser::OnTabClosing(WebContents* contents) { void Browser::OnTabClosing(WebContents* contents) {
......
...@@ -92,5 +92,4 @@ void ManifestWebAppBrowserController::OnTabInserted( ...@@ -92,5 +92,4 @@ void ManifestWebAppBrowserController::OnTabInserted(
if (app_launch_url_.is_empty()) if (app_launch_url_.is_empty())
app_launch_url_ = contents->GetURL(); app_launch_url_ = contents->GetURL();
AppBrowserController::OnTabInserted(contents); AppBrowserController::OnTabInserted(contents);
UpdateToolbarVisibility(false);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment