Map fullscreen button to maximize

chrome/browser/ui/ash/chrome_shell_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/ash/chrome_shell_delegate.h"
 
 #include "ash/launcher/launcher_types.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/wm/window_util.h"
 #include "base/command_line.h"
 #include "chrome/browser/chromeos/login/screen_locker.h"
 #include "chrome/browser/extensions/api/terminal/terminal_extension_helper.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/sessions/tab_restore_service.h"
 #include "chrome/browser/sessions/tab_restore_service_factory.h"
 #include "chrome/browser/ui/app_list/app_list_view_delegate.h"
 #include "chrome/browser/ui/ash/app_list/app_list_controller_ash.h"
 #include "chrome/browser/ui/ash/caps_lock_handler.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
 #include "chrome/browser/ui/ash/user_action_handler.h"
 #include "chrome/browser/ui/ash/window_positioner.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/views/frame/browser_view.h"

[tfarina, 2012/09/12 14:47:51]

no, no, no. You can't include this anymore!

[sschmitz, 2012/09/12 23:45:50]

Thanks. Found a better way.
Done

 #include "chrome/browser/ui/webui/chrome_web_contents_handler.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/time_format.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 #include "grit/generated_resources.h"
 #include "ui/aura/client/user_action_client.h"
@@ skipping 48 matching lines @@
       chrome::NOTIFICATION_SESSION_STARTED,
       content::NotificationService::AllSources());
 #endif
 }
 
 ChromeShellDelegate::~ChromeShellDelegate() {
   if (instance_ == this)
     instance_ = NULL;
 }
 
+void ChromeShellDelegate::RemoveFullScreenExitBubble() {
+  Browser* browser = GetTargetBrowser();
+  if (!browser)
+    return;
+  BrowserView* browser_view = BrowserView::GetBrowserViewForBrowser(browser);
+  if (!browser_view)
+    return;
+  browser_view->RemoveFullScreenExitBubble();
+}
+
 bool ChromeShellDelegate::IsUserLoggedIn() {
 #if defined(OS_CHROMEOS)
   // When running a Chrome OS build outside of a device (i.e. on a developer's
   // workstation) and not running as login-manager, pretend like we're always
   // logged in.
   if (!base::chromeos::IsRunningOnChromeOS() &&
       !CommandLine::ForCurrentProcess()->HasSwitch(switches::kLoginManager)) {
     return true;
   }
 
@@ skipping 237 matching lines @@
   return new UserActionHandler;
 }
 
 void ChromeShellDelegate::OpenFeedbackPage() {
   chrome::OpenFeedbackDialog(GetTargetBrowser());
 }
 
 void ChromeShellDelegate::RecordUserMetricsAction(
     ash::UserMetricsAction action) {
   switch (action) {
+    case ash::UMA_ACCEL_MAXIMIZE_RESTORE:
+      // TODO(sschmitz): Using the old name "Accel_Fullscreen_F4". Should we
+      // change it to Accel_Maximize_Restore_F4?
+      content::RecordAction(content::UserMetricsAction("Accel_Fullscreen_F4"));
+      break;
     case ash::UMA_ACCEL_PREVWINDOW_TAB:
       content::RecordAction(content::UserMetricsAction("Accel_PrevWindow_Tab"));
       break;
     case ash::UMA_ACCEL_NEXTWINDOW_TAB:
       content::RecordAction(content::UserMetricsAction("Accel_NextWindow_Tab"));
       break;
     case ash::UMA_ACCEL_PREVWINDOW_F5:
       content::RecordAction(content::UserMetricsAction("Accel_PrevWindow_F5"));
       break;
     case ash::UMA_ACCEL_NEXTWINDOW_F5:
@@ skipping 55 matching lines @@
       ash::Shell::GetInstance()->ShowLauncher();
       break;
     default:
       NOTREACHED() << "Unexpected notification " << type;
   }
 #else
   // MSVC++ warns about switch statements without any cases.
   NOTREACHED() << "Unexpected notification " << type;
 #endif
 }