Move most visited item state info from InstantController to InstantService.

chrome/browser/ui/search/instant_page_unittest.cc

 // 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.
 
 #include "chrome/browser/ui/search/instant_page.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
@@ skipping 29 matching lines @@
                     int height,
                     InstantSizeUnits units));
   MOCK_METHOD2(FocusOmnibox,
                void(const content::WebContents* contents,
                     OmniboxFocusState state));
   MOCK_METHOD4(NavigateToURL,
                void(const content::WebContents* contents,
                     const GURL& url,
                     content::PageTransition transition,
                     WindowOpenDisposition disposition));
-  MOCK_METHOD1(DeleteMostVisitedItem,
-               void(InstantRestrictedID most_visited_item_id));
+  MOCK_METHOD1(DeleteMostVisitedItem, void(const GURL& item_url));

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

It's a little pedantic, but you could test that OnMessageReceived() dispatches
calls to these mocks.

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

Done.

   MOCK_METHOD1(UndoMostVisitedDeletion,
-               void(InstantRestrictedID most_visited_item_id));
+               void(const GURL& most_visited_item_url));
   MOCK_METHOD0(UndoAllMostVisitedDeletions, void());
   MOCK_METHOD1(InstantPageLoadFailed, void(content::WebContents* contents));
 };
 
 class FakePage : public InstantPage {
  public:
   FakePage(Delegate* delegate, const std::string& instant_url)
     : InstantPage(delegate, instant_url) {
   }
   virtual ~FakePage() {
@@ skipping 40 matching lines @@
   page->SetContents(web_contents());
   NavigateAndCommit(GURL("http://example.com/"));
   EXPECT_FALSE(page->IsLocal());
   process()->sink().ClearMessages();
   page->DetermineIfPageSupportsInstant();
   const IPC::Message* message = process()->sink().GetFirstMessageMatching(
       ChromeViewMsg_DetermineIfPageSupportsInstant::ID);
   ASSERT_TRUE(message != NULL);
   EXPECT_EQ(web_contents()->GetRoutingID(), message->routing_id());
 }