Fixing a regression introduced with r69101, which now prevents Chrome Frame...

chrome_frame/chrome_frame_npapi.cc

Index: chrome_frame/chrome_frame_npapi.cc
===================================================================
--- chrome_frame/chrome_frame_npapi.cc	(revision 69128)
+++ chrome_frame/chrome_frame_npapi.cc	(working copy)
@@ -11,6 +11,7 @@
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
+#include "chrome/common/url_constants.h"
 #include "chrome/test/automation/tab_proxy.h"
 #include "chrome_frame/ff_privilege_check.h"
 #include "chrome_frame/np_utils.h"
@@ -418,6 +419,19 @@
       print_bounds);
 }
 
+bool ChromeFrameNPAPI::IsSchemeAllowed(const GURL& url) {

[amit, 2010/12/14 19:37:58]

This looks identical to the ActiveX version. So maybe move this down to the base
class and have a dcheck(!is_privileged_) in the ActiveDoc::IsSchemeAllowed(...)
for now?

+  bool allowed = NavigationConstraintsImpl::IsSchemeAllowed(url);
+  if (allowed)
+    return true;
+
+  if (is_privileged_ &&
+      (url.SchemeIs(chrome::kDataScheme) ||
+       url.SchemeIs(chrome::kExtensionScheme))) {
+    return true;
+  }
+  return false;
+}
+
 void ChromeFrameNPAPI::UrlNotify(const char* url, NPReason reason,
                                  void* notify_data) {
   if (enabled_popups_) {