Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2195)

Unified Diff: chrome/browser/ui/cocoa/browser_window_cocoa.mm

Issue 2781633003: Allow keyboard shortcuts can be captured by webpage when toolbar is not visible (Closed)
Patch Set: Resolve review comments Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/browser_window_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
index 2991e80ab6535094deb95aa54984a6c72c13f3b9..675cee67e9f2fc94fec1a39165f82fb8f4a80146 100644
--- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
+++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
@@ -322,6 +322,15 @@ NSString* BrowserWindowCocoa::WindowTitle() {
browser_->GetWindowTitleForCurrentTab(include_app_name));
}
+bool BrowserWindowCocoa::IsToolbarShowing() const {
+ if (!IsFullscreen())
+ return true;
+
+ // TODO(zijiehe): Retrieve the visibility of toolbar from
+ // FullscreenToolbarController. See http://crbug.com/702251.
msw 2017/04/05 23:55:11 This looks like the wrong bug? "Event.preventDefau
Hzj_jie 2017/04/06 02:37:59 Maybe I can add both bugs to the comment. The clos
+ return true;
+}
+
void BrowserWindowCocoa::BookmarkBarStateChanged(
BookmarkBar::AnimateChangeType change_type) {
[[controller_ bookmarkBarController]

Powered by Google App Engine
This is Rietveld 408576698