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

chrome_frame/chrome_frame_delegate.h

Index: chrome_frame/chrome_frame_delegate.h
===================================================================
--- chrome_frame/chrome_frame_delegate.h	(revision 68604)
+++ chrome_frame/chrome_frame_delegate.h	(working copy)
@@ -212,4 +212,23 @@
   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_