Fixed behavior of the bookmark bar visibility.

chrome/browser/ui/webui/chrome_web_ui.h

Index: chrome/browser/ui/webui/chrome_web_ui.h
diff --git a/chrome/browser/ui/webui/chrome_web_ui.h b/chrome/browser/ui/webui/chrome_web_ui.h
index 765fe0300f72614010e90eaeab0368657964fcd2..c4aaae4d04f7a2cfbcb028dc186e15467bb121f6 100644
--- a/chrome/browser/ui/webui/chrome_web_ui.h
+++ b/chrome/browser/ui/webui/chrome_web_ui.h
@@ -18,11 +18,8 @@ class ChromeWebUI : public WebUI {
   // Returns the profile for this WebUI.
   Profile* GetProfile() const;
 
-  // Returns true if the bookmark bar should be forced to being visible,
-  // overriding the user's preference.
-  bool force_bookmark_bar_visible() const {
-    return force_bookmark_bar_visible_;
-  }
+  // Returns true if the bookmark bar should be visible, inlined in the webui.

[sky, 2011/10/05 16:59:12]

What does 'inlined in webui' mean here? Is that part of the sentence needed?

[Joao da Silva, 2011/10/06 19:45:38]

It refers to the detached state. Made it clearer in the comment.

+  virtual bool IsBookmarkBarVisible() const;
 
   // IsMoreWebUI returns a command line flag that tracks whether to use
   // available WebUI implementations of native dialogs.
@@ -32,14 +29,7 @@ class ChromeWebUI : public WebUI {
   // WebUI.
   static void OverrideMoreWebUI(bool use_more_webui);
 
- protected:
-  void set_force_bookmark_bar_visible(bool value) {
-    force_bookmark_bar_visible_ = value;
-  }
-
  private:
-  bool force_bookmark_bar_visible_;
-
   DISALLOW_COPY_AND_ASSIGN(ChromeWebUI);
 };