[subresource_filter] add //chrome level unit test harness

chrome/browser/subresource_filter/chrome_subresource_filter_client.h

 // Copyright 2016 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_SUBRESOURCE_FILTER_CHROME_SUBRESOURCE_FILTER_CLIENT_H_
 #define CHROME_BROWSER_SUBRESOURCE_FILTER_CHROME_SUBRESOURCE_FILTER_CLIENT_H_
 
 #include "base/macros.h"
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/subresource_filter/content/browser/subresource_filter_client.h"
@@ skipping 52 matching lines @@
   explicit ChromeSubresourceFilterClient(content::WebContents* web_contents);
   ~ChromeSubresourceFilterClient() override;
 
   // SubresourceFilterClient:
   void ToggleNotificationVisibility(bool visibility) override;
   bool IsWhitelistedByContentSettings(const GURL& url) override;
   void WhitelistByContentSettings(const GURL& url) override;
   subresource_filter::VerifiedRulesetDealer::Handle* GetRulesetDealer()
       override;
 
+  bool shown_for_navigation() { return shown_for_navigation_; }

[engedy, 2017/04/20 12:34:33]

nit: How about {did_show|shown}_ui_for_navigation()? I think Tanja might be
renaming the underlying member variable in a concurrent CL.

[Charlie Harrison, 2017/04/20 15:55:20]

Done, I've also changed the underlying member to be consistent.

+
   static void LogAction(SubresourceFilterAction action);
 
  private:
   ContentSetting GetContentSettingForUrl(const GURL& url);
   content::WebContents* web_contents_;
   bool shown_for_navigation_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeSubresourceFilterClient);
 };
 
 #endif  // CHROME_BROWSER_SUBRESOURCE_FILTER_CHROME_SUBRESOURCE_FILTER_CLIENT_H_