[subresource_filter] add //chrome level unit test harness

chrome/browser/subresource_filter/chrome_subresource_filter_client.h

Index: chrome/browser/subresource_filter/chrome_subresource_filter_client.h
diff --git a/chrome/browser/subresource_filter/chrome_subresource_filter_client.h b/chrome/browser/subresource_filter/chrome_subresource_filter_client.h
index e706577dff44b7d68c78d3330bb3144748e32cd8..e0cfcc55a74b991100d5ffe9e0cac3260f6e217e 100644
--- a/chrome/browser/subresource_filter/chrome_subresource_filter_client.h
+++ b/chrome/browser/subresource_filter/chrome_subresource_filter_client.h
@@ -70,6 +70,8 @@ class ChromeSubresourceFilterClient
   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: