Move most visited item state info from InstantController to InstantService.

chrome/browser/ui/search/instant_page.h

 // Copyright 2013 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_SEARCH_INSTANT_PAGE_H_
 #define CHROME_BROWSER_UI_SEARCH_INSTANT_PAGE_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 65 matching lines @@
     // Called when the page wants to navigate to |url|. Usually used by the
     // page to navigate to privileged destinations (e.g. chrome:// URLs) or to
     // navigate to URLs that are hidden from the page using Restricted IDs (rid
     // in the API).
     virtual void NavigateToURL(const content::WebContents* contents,
                                const GURL& url,
                                content::PageTransition transition,
                                WindowOpenDisposition disposition) = 0;
 
     // Called when the SearchBox wants to delete a Most Visited item.
-    virtual void DeleteMostVisitedItem(
-        InstantRestrictedID most_visited_item_id) = 0;
+    virtual void DeleteMostVisitedItem(const GURL& item_url) = 0;
 
     // Called when the SearchBox wants to undo a Most Visited deletion.
-    virtual void UndoMostVisitedDeletion(
-        InstantRestrictedID most_visited_item_id) = 0;
+    virtual void UndoMostVisitedDeletion(const GURL& most_visited_item_url) = 0;
 
     // Called when the SearchBox wants to undo all Most Visited deletions.
     virtual void UndoAllMostVisitedDeletions() = 0;
 
     // Called when the page fails to load for whatever reason.
     virtual void InstantPageLoadFailed(content::WebContents* contents) = 0;
 
    protected:
     virtual ~Delegate();
   };
@@ skipping 142 matching lines @@
                         const std::vector<InstantSuggestion>& suggestions);
   void OnInstantSupportDetermined(int page_id, bool supports_instant);
   void OnShowInstantOverlay(int page_id,
                             int height,
                             InstantSizeUnits units);
   void OnFocusOmnibox(int page_id, OmniboxFocusState state);
   void OnSearchBoxNavigate(int page_id,
                            const GURL& url,
                            content::PageTransition transition,
                            WindowOpenDisposition disposition);
-  void OnDeleteMostVisitedItem(InstantRestrictedID most_visited_item_id);
-  void OnUndoMostVisitedDeletion(InstantRestrictedID most_visited_item_id);
+  void OnDeleteMostVisitedItem(const GURL& item_url);

[Jered, 2013/05/21 20:18:21]

Just "url" is fine here and below, since the method name makes it obvious what
the url is for.

[kmadhusu, 2013/05/23 18:26:22]

Done.

+  void OnUndoMostVisitedDeletion(const GURL& most_visited_item_url);
   void OnUndoAllMostVisitedDeletions();
 
   Delegate* const delegate_;
   const std::string instant_url_;
   bool supports_instant_;
   bool instant_support_determined_;
 
   DISALLOW_COPY_AND_ASSIGN(InstantPage);
 };
 
 #endif  // CHROME_BROWSER_UI_SEARCH_INSTANT_PAGE_H_