crbug.com/127841 - Request Tablet Site on CB with touch screen.

chrome/browser/ui/browser_commands.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/browser_commands.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_editor.h"
@@ skipping 44 matching lines @@
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/devtools_agent_host_registry.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/page_navigator.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
+#include "content/public/common/content_client.h"
 #include "content/public/common/content_restriction.h"
 #include "content/public/common/renderer_preferences.h"
 #include "content/public/common/url_constants.h"
 #include "net/base/escape.h"
+#include "webkit/glue/user_agent.h"
 #include "webkit/glue/webkit_glue.h"
 
 #if defined(OS_MACOSX)
 #include "ui/base/cocoa/find_pasteboard.h"
 #endif
 
 #if defined(OS_WIN)
 #include "base/win/metro.h"
 #endif
 
@@ skipping 778 matching lines @@
 
 void OpenUpdateChromeDialog(Browser* browser) {
   content::RecordAction(UserMetricsAction("UpdateChrome"));
   browser->window()->ShowUpdateChromeDialog();
 }
 
 void ToggleSpeechInput(Browser* browser) {
   GetActiveWebContents(browser)->GetRenderViewHost()->ToggleSpeechInput();
 }
 
+void ToggleRequestTabletSite(Browser* browser)
+{
+  WebContents* current_tab = GetActiveWebContents(browser);
+  if (!current_tab)
+    return; // The control is disabled in this case

[gone, 2012/08/03 20:13:26]

Maybe put a general comment above about the control being disabled if we return
early instead of repeating?

[sschmitz, 2012/08/03 21:20:51]

Done.

+  NavigationController& controller = current_tab->GetController();
+  NavigationEntry* entry = controller.GetActiveEntry();
+  if (!entry)
+    return; // The control is disabled (dim) in this case
+  if (entry->GetIsOverridingUserAgent()) {
+    entry->SetIsOverridingUserAgent(false);
+    const int entryCount = controller.GetEntryCount();
+    for (int ii = 0; ii < entryCount; ++ii) {
+      NavigationEntry* entryAtIdx = controller.GetEntryAtIndex(ii);

[gone, 2012/08/03 20:13:26]

Can't remember what the style guide says exactly, but IIRC we use _ instead of
camel case for variable names.  entryAtIdx -> entry_at_index,
entryCount->entry_count, etc.

[sschmitz, 2012/08/03 21:20:51]

Done.

+      if (entryAtIdx) entryAtIdx->SetIsOverridingUserAgent(false);
+    }
+    current_tab->SetUserAgentOverride(std::string());
+  } else {
+    entry->SetIsOverridingUserAgent(true);
+    const int entryCount = controller.GetEntryCount();
+    for (int ii = 0; ii < entryCount; ++ii) {
+      NavigationEntry* entryAtIdx = controller.GetEntryAtIndex(ii);
+      if (entryAtIdx) entryAtIdx->SetIsOverridingUserAgent(true);
+    }
+    current_tab->SetUserAgentOverride(
+        webkit_glue::BuildUserAgentOverrideForTabletSiteFromUserAgent(
+            content::GetUserAgent(entry->GetOriginalRequestURL())));
+  }
+  controller.Reload(true);
+}
+
 void ToggleFullscreenMode(Browser* browser) {
   browser->fullscreen_controller()->ToggleFullscreenMode();
 }
 
 void ClearCache(Browser* browser) {
   BrowsingDataRemover* remover = new BrowsingDataRemover(browser->profile(),
       BrowsingDataRemover::EVERYTHING,
       base::Time());
   remover->Remove(BrowsingDataRemover::REMOVE_CACHE,
                   BrowsingDataHelper::UNPROTECTED_WEB);
@@ skipping 112 matching lines @@
   if (!tab_contents)
     tab_contents = new TabContents(contents);
   app_browser->tab_strip_model()->AppendTabContents(tab_contents, true);
 
   contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   contents->GetRenderViewHost()->SyncRendererPrefs();
   app_browser->window()->Show();
 }
 
 }  // namespace chrome