Also filter the URLs from incoming non-provisional navigation IPCs.

content/browser/web_contents/test_web_contents.h

 // Copyright (c) 2012 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 CONTENT_BROWSER_WEB_CONTENTS_TEST_WEB_CONTENTS_H_
 #define CONTENT_BROWSER_WEB_CONTENTS_TEST_WEB_CONTENTS_H_
 
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/test/web_contents_tester.h"
@@ skipping 72 matching lines @@
   void ExpectSetHistoryLengthAndPrune(const SiteInstance* site_instance,
                                       int history_length,
                                       int32 min_page_id);
 
   // Compares the arguments passed in with the expected arguments passed in
   // to |ExpectSetHistoryLengthAndPrune()|.
   virtual void SetHistoryLengthAndPrune(const SiteInstance* site_instance,
                                         int history_length,
                                         int32 min_page_id) OVERRIDE;
 
+  void TestDidFinishLoad(int64 frame_id, const GURL& url, bool is_main_frame);

[Charlie Reis, 2013/01/07 20:58:46]

You can probably skip this with a FRIEND_TEST_ALL_PREFIXES line in
web_contents_impl.h, since there's several of those already.  That will let your
test call OnDidFinishLoad directly.

[jochen (gone - plz use gerrit), 2013/01/07 21:11:59]

As discussed, I think emulating an IPC is a cleaner approach here, because
otherwise I'd need to setup the message_source_ variable which is otherwise set
by OnMessageReceived. It feels like this hardcodes unnecessarily much knowledge
about the internal implementation

+  void TestDidFailLoadWithError(int64 frame_id,
+                                const GURL& url,
+                                bool is_main_frame,
+                                int error_code,
+                                const string16& error_description);
+
  protected:
   // The deprecated WebContentsTester still needs to subclass this.
   explicit TestWebContents(BrowserContext* browser_context);
 
  private:
   // WebContentsImpl overrides
   virtual void CreateNewWindow(
       int route_id,
       const ViewHostMsg_CreateWindow_Params& params,
       SessionStorageNamespace* session_storage_namespace) OVERRIDE;
@@ skipping 15 matching lines @@
   bool expect_set_history_length_and_prune_;
   scoped_refptr<const SiteInstanceImpl>
     expect_set_history_length_and_prune_site_instance_;
   int expect_set_history_length_and_prune_history_length_;
   int32 expect_set_history_length_and_prune_min_page_id_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_WEB_CONTENTS_TEST_WEB_CONTENTS_H_