Disable encoding menu item for chrome://bookmarks

chrome/browser/ui/browser.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_H_
 #define CHROME_BROWSER_UI_BROWSER_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 911 matching lines @@
   virtual void CommitInstant(TabContentsWrapper* preview_contents) OVERRIDE;
   virtual void SetSuggestedText(const string16& text,
                                 InstantCompleteBehavior behavior) OVERRIDE;
   virtual gfx::Rect GetInstantBounds() OVERRIDE;
 
   // Command and state updating ///////////////////////////////////////////////
 
   // Initialize state for all browser commands.
   void InitCommandState();
 
+  // Returns |true| if entry has an internal chrome:// URL.
+  bool HasInternalURL(const NavigationEntry *entry) const;

[brettw, 2011/07/06 16:55:24]

It looks like this doesn't actually have to be a member. Can you make it
function in an anon namespace in browser.cc to reduce the scope (browser.h is
too large).

[Alexei Svitkine (slow), 2011/07/06 19:00:05]

Done.

+
   // Update commands whose state depends on the tab's state.
   void UpdateCommandsForTabState();
 
   // Updates commands when the content's restrictions change.
   void UpdateCommandsForContentRestrictionState();
 
   // Updates commands for enabling developer tools.
   void UpdateCommandsForDevTools();
 
   // Updates commands for bookmark editing.
@@ skipping 322 matching lines @@
 
   scoped_ptr<InstantController> instant_;
   scoped_ptr<InstantUnloadHandler> instant_unload_handler_;
 
   BookmarkBar::State bookmark_bar_state_;
 
   DISALLOW_COPY_AND_ASSIGN(Browser);
 };
 
 #endif  // CHROME_BROWSER_UI_BROWSER_H_