Cleanup: Fix some out of date comments.

chrome/test/automation/browser_proxy.h

 // Copyright (c) 2010 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_TEST_AUTOMATION_BROWSER_PROXY_H_
 #define CHROME_TEST_AUTOMATION_BROWSER_PROXY_H_
 #pragma once
 
-#include "build/build_config.h"
-
-#if defined(OS_WIN)
-#include <windows.h>
-#endif
-
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/content_settings.h"
 #include "chrome/test/automation/automation_handle_tracker.h"
 
+class AutocompleteEditProxy;
 class GURL;
 class TabProxy;
 class WindowProxy;
-class AutocompleteEditProxy;
 
 namespace gfx {
   class Point;
   class Rect;
 }
 
 // This class presents the interface to actions that can be performed on
 // a given browser window.  Note that this object can be invalidated at any
 // time if the corresponding browser window in the app is closed.  In that case,
 // any subsequent calls will return false immediately.
@@ skipping 12 matching lines @@
   // success.
   bool BringToFront() WARN_UNUSED_RESULT;
 
   // Checks to see if a command is enabled or not. If the call was successful,
   // puts the result in |enabled| and returns true.
   bool IsMenuCommandEnabled(int id, bool* enabled) WARN_UNUSED_RESULT;
 
   // Append a new tab to the TabStrip.  The new tab is selected.
   // The new tab navigates to the given tab_url.
   // Returns true if successful.
-  // TODO(mpcomplete): If the navigation results in an auth challenge, the
-  // TabProxy we attach won't know about it.  See bug 666730.
   bool AppendTab(const GURL& tab_url) WARN_UNUSED_RESULT;
 
   // Gets the (zero-based) index of the currently active tab. Returns true if
   // successful.
   bool GetActiveTabIndex(int* active_tab_index) const WARN_UNUSED_RESULT;
 
   // Returns the number of tabs in the given window.  Returns true if
   // the call was successful.
   bool GetTabCount(int* num_tabs) const WARN_UNUSED_RESULT;
 
@@ skipping 90 matching lines @@
 
   // Editing of the bookmark model.  Bookmarks are referenced by id.
   // Bookmark or group (folder) creation:
   bool AddBookmarkGroup(int64 parent_id, int index,
                         std::wstring& title) WARN_UNUSED_RESULT;
   bool AddBookmarkURL(int64 parent_id, int index,
                       std::wstring& title, const GURL& url) WARN_UNUSED_RESULT;
   // Bookmark editing:
   bool ReparentBookmark(int64 id, int64 new_parent_id,
                         int index) WARN_UNUSED_RESULT;
-  bool SetBookmarkTitle(int64 id, std::wstring& title) WARN_UNUSED_RESULT;
+  bool SetBookmarkTitle(int64 id, const std::wstring& title) WARN_UNUSED_RESULT;
   bool SetBookmarkURL(int64 id, const GURL& url) WARN_UNUSED_RESULT;
   // Finally, bookmark deletion:
   bool RemoveBookmark(int64 id) WARN_UNUSED_RESULT;
 
   // Fills |*is_visible| with whether the browser's download shelf is currently
   // visible. The return value indicates success. On failure, |*is_visible| is
   // unchanged.
   bool IsShelfVisible(bool* is_visible) WARN_UNUSED_RESULT;
 
   // Shows or hides the download shelf.
@@ skipping 60 matching lines @@
                            std::vector<float>* stop_times);
 
 
  protected:
   virtual ~BrowserProxy() {}
  private:
   DISALLOW_COPY_AND_ASSIGN(BrowserProxy);
 };
 
 #endif  // CHROME_TEST_AUTOMATION_BROWSER_PROXY_H_