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

Unified Diff: chrome/browser/ui/browser.cc

Issue 386463002: Remove --disable-fullscreen-within-tab command-line flag. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/tab_contents/tab_contents_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index db19aee5d85607283b8c7f1bf17e173b15e94a7a..a2bcd0ba8249dd4eceb3c203584b41446735e13b 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1602,8 +1602,7 @@ void Browser::EnumerateDirectory(WebContents* web_contents,
}
bool Browser::EmbedsFullscreenWidget() const {
Avi (use Gerrit) 2014/07/10 05:30:04 Is this a useful function if it always returns tru
miu 2014/07/10 06:20:16 Can't kill it. It's an override of WebContentsDel
Avi (use Gerrit) 2014/07/10 13:14:22 No problem.
- return !CommandLine::ForCurrentProcess()->
- HasSwitch(switches::kDisableFullscreenWithinTab);
+ return true;
}
void Browser::ToggleFullscreenModeForTab(WebContents* web_contents,
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/tab_contents/tab_contents_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698