Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3104)

Unified Diff: chrome/renderer/prerender/prerender_dispatcher.h

Issue 10424002: Adding unit tests for PrerenderDispatcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleaner interface for more readable tests Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/prerender/prerender_dispatcher.h
diff --git a/chrome/renderer/prerender/prerender_dispatcher.h b/chrome/renderer/prerender/prerender_dispatcher.h
index 3ee2b7f3217cb3edea22dd9f79ed2ee9d8ab89b6..35f0093a7a5ebc6728a3dcc8ec02fc09c8d9a354 100644
--- a/chrome/renderer/prerender/prerender_dispatcher.h
+++ b/chrome/renderer/prerender/prerender_dispatcher.h
@@ -10,6 +10,7 @@
#include "base/compiler_specific.h"
#include "base/memory/scoped_ptr.h"
+#include "base/gtest_prod_util.h"
#include "content/public/renderer/render_process_observer.h"
class GURL;
@@ -31,6 +32,16 @@ class PrerenderDispatcher : public content::RenderProcessObserver {
bool IsPrerenderURL(const GURL & url) const;
private:
+ friend class PrerenderDispatcherTest;
+ FRIEND_TEST_ALL_PREFIXES(PrerenderDispatcherTest, PrerenderDispatcherEmpty);
gavinp 2012/05/24 17:39:40 Do you need these now? I think you don't, and pro
dominich 2012/05/24 18:11:56 Done.
+ FRIEND_TEST_ALL_PREFIXES(PrerenderDispatcherTest,
+ PrerenderDispatcherSingleAdd);
+ FRIEND_TEST_ALL_PREFIXES(PrerenderDispatcherTest,
+ PrerenderDispatcherMultipleAdd);
+ FRIEND_TEST_ALL_PREFIXES(PrerenderDispatcherTest,
+ PrerenderDispatcherSingleRemove);
+ FRIEND_TEST_ALL_PREFIXES(PrerenderDispatcherTest,
+ PrerenderDispatcherMultipleRemove);
void OnAddPrerenderURL(const GURL& url);
void OnRemovePrerenderURL(const GURL& url);

Powered by Google App Engine
This is Rietveld 408576698