Add support for gcf:about:plugins in chrome frame full tab mode. The URL vali...

chrome_frame/chrome_frame_delegate.h

 // Copyright (c) 2010 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_FRAME_CHROME_FRAME_DELEGATE_H_
 #define CHROME_FRAME_CHROME_FRAME_DELEGATE_H_
 
 #include <atlbase.h>
 #include <atlwin.h>
 #include <queue>
@@ skipping 194 matching lines @@
       return true;
     }
 
     return false;
   }
 
   Lock lock_;
   std::queue<Task*> pending_tasks_;
 };
 
+// Provides an interface which controls navigation within ChromeFrame.
+class NavigationConstraints {
+ public:
+  virtual bool AllowUnsafeUrls() = 0;
+  virtual bool IsSchemeAllowed(const GURL&url) = 0;

[amit, 2010/12/10 23:56:17]

nit: spacing after &

[ananta, 2010/12/11 02:11:11]

Done and moved these classes to the navigation_constraints.h/.cc files

+  virtual bool IsZoneAllowed(const GURL& url) = 0;
+};
+
+// Provides default implementation for the NavigationConstraints interface.
+class NavigationConstraintsImpl : public NavigationConstraints {

[amit, 2010/12/10 23:56:17]

why not move this into its own source file and include IInternetSecurityManager
in the implementation? 

[ananta, 2010/12/11 02:11:11]

I thought about that. Decided against that for the timebeing as the only clients
of the IInternetSecurityManager interface are the activex and active document
combination. 

+ public:
+  virtual ~NavigationConstraintsImpl() {}
+
+  // NavigationConstraints method overrides.
+  virtual bool AllowUnsafeUrls();
+  virtual bool IsSchemeAllowed(const GURL&url);
+  virtual bool IsZoneAllowed(const GURL& url);
+};
+
 #endif  // CHROME_FRAME_CHROME_FRAME_DELEGATE_H_