Drive: Hide the URL on opening files on Drive.

chrome/browser/ui/toolbar/toolbar_model.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/toolbar/toolbar_model.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autocomplete/autocomplete_input.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 69 matching lines @@
     if (entry->IsViewSourceMode() ||
         entry->GetPageType() == content::PAGE_TYPE_INTERSTITIAL) {
       return true;
     }
   }
 
   WebContents* web_contents = delegate_->GetActiveWebContents();
   if (web_contents && web_contents->GetWebUIForCurrentState())
     return !web_contents->GetWebUIForCurrentState()->ShouldHideURL();
 
-  if (entry && entry->GetURL().SchemeIs(chrome::kExtensionScheme))
+  if (entry && (entry->GetURL().SchemeIs(chrome::kExtensionScheme) ||
+#if defined(OS_CHROMEOS)

[Aaron Boodman, 2012/08/10 04:36:56]

I'm not a fan of mixing runtime and compile time conditionals this way. I'd
prefer just having a separate if statement that was entirely guarded by
OS_CHROMEOS.

[yoshiki, 2012/08/10 05:34:34]

Done.

+                entry->GetURL().SchemeIs(chrome::kDriveScheme)
+#endif
+      )) {
     return false;
+  }
 
   return true;
 }
 
 ToolbarModel::SecurityLevel ToolbarModel::GetSecurityLevel() const {
   if (input_in_progress_)  // When editing, assume no security style.
     return NONE;
 
   NavigationController* navigation_controller = GetNavigationController();
   if (!navigation_controller)  // We might not have a controller on init.
@@ skipping 67 matching lines @@
       UTF8ToUTF16(cert.subject().country_name));
 }
 
 NavigationController* ToolbarModel::GetNavigationController() const {
   // This |current_tab| can be NULL during the initialization of the
   // toolbar during window creation (i.e. before any tabs have been added
   // to the window).
   WebContents* current_tab = delegate_->GetActiveWebContents();
   return current_tab ? &current_tab->GetController() : NULL;
 }