Add implementation for Ad Networks Database

chrome/browser/extensions/activity_log/ad_injection_browsertest.cc

Index: chrome/browser/extensions/activity_log/ad_injection_browsertest.cc
diff --git a/chrome/browser/extensions/activity_log/ad_injection_browsertest.cc b/chrome/browser/extensions/activity_log/ad_injection_browsertest.cc
index 2df700d1628031ccf2761a79467d12487857dd43..608afac2a22273e43f7e509cecb8e684ff3c537b 100644
--- a/chrome/browser/extensions/activity_log/ad_injection_browsertest.cc
+++ b/chrome/browser/extensions/activity_log/ad_injection_browsertest.cc
@@ -91,9 +91,12 @@ class TestAdNetworkDatabase : public AdNetworkDatabase {
  public:
   TestAdNetworkDatabase();
   virtual ~TestAdNetworkDatabase();
-  virtual bool IsAdNetwork(const GURL& url) const OVERRIDE;
 
  private:

[felt, 2014/04/23 03:01:36]

Is there any reason why the browsertest has different public/private than the
parent class?

[Devlin, 2014/04/23 18:16:55]

Yeah, it's a C++ trick that we commonly do in order to make sure that the only
time these methods are called is through the original implementation.  You can
see it frequently with classes implementing observers, so that only the
notifying class calls them.

+  // AdNetworkDatabase implementation.
+  virtual bool IsEnabled() const OVERRIDE { return true; }
+  virtual bool IsAdNetwork(const GURL& url) const OVERRIDE;
+
   GURL ad_network_url_;
 };