Support manual (white|black)list, previewing and allowing after interstitial

chrome/browser/managed_mode/managed_mode_resource_throttle.h

 // Copyright (c) 2012 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_MANAGED_MODE_MANAGED_MODE_RESOURCE_THROTTLE_H_
 #define CHROME_BROWSER_MANAGED_MODE_MANAGED_MODE_RESOURCE_THROTTLE_H_
 
 #include "base/compiler_specific.h"
 #include "base/memory/weak_ptr.h"
 #include "content/public/browser/resource_throttle.h"
 
 class ManagedModeURLFilter;
 
 namespace net {
 class URLRequest;
 }
 
-namespace policy {
-class URLBlacklist;
-}
-
 class ManagedModeResourceThrottle : public content::ResourceThrottle {
  public:
   ManagedModeResourceThrottle(const net::URLRequest* request,
                               int render_process_host_id,
                               int render_view_id,
                               bool is_main_frame);
   virtual ~ManagedModeResourceThrottle();
 
+  // Adds/removes a temporary exception to filtering for a
+  // render_process_host_id and render_view_id pair to the preview map.
+  // Adding saves the last approved hostname in the map.
+  static void AddTemporaryException(int render_process_host_id,
+                                    int render_view_id,
+                                    const std::string& host);
+  static void RemoveTemporaryException(int render_process_host_id,
+                                       int render_view_id);
+
+  // Checks whether the |url| needs to trigger an interstitial. If it

[Bernhard Bauer, 2012/11/27 18:44:46]

Update the comment now? We do more than check, also the description of the logic
isn't fully accurate. Maybe we can just leave it out (we have a description in
the implementation)?

[Sergiu, 2012/11/28 12:53:25]

Done.

+  // |is_redirect| or the user already saw an interstitial on the same hostname
+  // as this |url| then do not show another one.
+  void ShowInterstitialIfNeeded(bool is_redirect,

[Bernhard Bauer, 2012/11/27 18:44:46]

Also, make this method private?

[Sergiu, 2012/11/28 12:53:25]

Done.

+                                const GURL& url,
+                                bool* defer);
+
   // content::ResourceThrottle implementation:
   virtual void WillStartRequest(bool* defer) OVERRIDE;
 
+  virtual void WillRedirectRequest(const GURL& new_url, bool* defer) OVERRIDE;
+
  private:
   void OnInterstitialResult(bool continue_request);
 
   base::WeakPtrFactory<ManagedModeResourceThrottle> weak_ptr_factory_;
   const net::URLRequest* request_;
   int render_process_host_id_;
   int render_view_id_;
   bool is_main_frame_;
+  bool after_interstitial_;
   const ManagedModeURLFilter* url_filter_;
 
   DISALLOW_COPY_AND_ASSIGN(ManagedModeResourceThrottle);
 };
 
 #endif  // CHROME_BROWSER_MANAGED_MODE_MANAGED_MODE_RESOURCE_THROTTLE_H_