Introduce subresource_filter::ConfigurationList and make querying it cheap.

components/subresource_filter/content/browser/content_subresource_filter_driver_factory.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 COMPONENTS_SUBRESOURCE_FILTER_CONTENT_BROWSER_CONTENT_SUBRESOURCE_FILTER_DRIVER_FACTORY_H_
 #define COMPONENTS_SUBRESOURCE_FILTER_CONTENT_BROWSER_CONTENT_SUBRESOURCE_FILTER_DRIVER_FACTORY_H_
 
 #include <map>
 #include <memory>
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/supports_user_data.h"
 #include "base/time/time.h"
 #include "components/safe_browsing_db/util.h"
 #include "components/subresource_filter/content/browser/content_subresource_filter_throttle_manager.h"
-#include "components/subresource_filter/core/browser/subresource_filter_features.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "ui/base/page_transition_types.h"
 #include "url/gurl.h"
 
 namespace content {
 class WebContents;
 }  // namespace content
 
 namespace safe_browsing {
 class SafeBrowsingServiceTest;
@@ skipping 82 matching lines @@
   void OnFirstSubresourceLoadDisallowed() override;
   bool ShouldSuppressActivation(
       content::NavigationHandle* navigation_handle) override;
   void WillProcessResponse(
       content::NavigationHandle* navigation_handle) override;
 
   ContentSubresourceFilterThrottleManager* throttle_manager() {
     return throttle_manager_.get();
   }
 
-  // TODO(https://crbug.com/708181): Allow tests to change the configuration
-  // after construction (which happens at WebContents creation) but before a
-  // navigation start. Can be removed once the Safe Browsing navigation throttle
-  // handles all activation decisions.
-  void set_configuration_for_testing(Configuration configuration) {
-    configuration_ = std::move(configuration);
-  }
-
   SubresourceFilterClient* client() { return client_.get(); }
 
  private:
   friend class ContentSubresourceFilterDriverFactoryTest;
   friend class safe_browsing::SafeBrowsingServiceTest;
 
   void ResetActivationState();
 
   // content::WebContentsObserver:
   void DidStartNavigation(
@@ skipping 10 matching lines @@
                                  ActivationList activation_list) const;
 
   void AddActivationListMatch(const GURL& url, ActivationList match_type);
   int CalculateHitPatternForActivationList(
       ActivationList activation_list) const;
   void RecordRedirectChainMatchPattern() const;
 
   void RecordRedirectChainMatchPatternForList(
       ActivationList activation_list) const;
 
-  Configuration configuration_;
-
   std::unique_ptr<SubresourceFilterClient> client_;
 
   std::unique_ptr<ContentSubresourceFilterThrottleManager> throttle_manager_;
 
   ActivationLevel activation_level_;
   ActivationDecision activation_decision_;
   bool measure_performance_;
 
   // The URLs in the navigation chain.
   std::vector<GURL> navigation_chain_;
 
   URLToActivationListsMap activation_list_matches_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentSubresourceFilterDriverFactory);
 };
 
 }  // namespace subresource_filter
 
 #endif  // COMPONENTS_SUBRESOURCE_FILTER_CONTENT_BROWSER_CONTENT_SUBRESOURCE_FILTER_DRIVER_FACTORY_H_