summaryrefslogtreecommitdiffstats
path: root/chrome_frame/chrome_frame_activex_base.h
diff options
context:
space:
mode:
authortommi@chromium.org <tommi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-10-01 16:00:21 +0000
committertommi@chromium.org <tommi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-10-01 16:00:21 +0000
commit10bf1a7030eeed02e2af95eaff012d2428ea5bc2 (patch)
treef381f0333bb9901aca6d2d5e2b0fdbfc5743b6d2 /chrome_frame/chrome_frame_activex_base.h
parent7f14e1fd31ddd510b19e2d9f6942c45435c9a8c6 (diff)
downloadchromium_src-10bf1a7030eeed02e2af95eaff012d2428ea5bc2.zip
chromium_src-10bf1a7030eeed02e2af95eaff012d2428ea5bc2.tar.gz
chromium_src-10bf1a7030eeed02e2af95eaff012d2428ea5bc2.tar.bz2
Committing patch 256001 for Roger:http://codereview.chromium.org/256001Allow Chrome Frame to display chrome-extension URLs when running in privilegedmode.BUG=noneTEST=see unit tests
Review URL: http://codereview.chromium.org/246050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27724 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/chrome_frame_activex_base.h')
-rw-r--r--chrome_frame/chrome_frame_activex_base.h18
1 files changed, 9 insertions, 9 deletions
diff --git a/chrome_frame/chrome_frame_activex_base.h b/chrome_frame/chrome_frame_activex_base.h
index bfdabb0..7754bc0 100644
--- a/chrome_frame/chrome_frame_activex_base.h
+++ b/chrome_frame/chrome_frame_activex_base.h
@@ -10,7 +10,7 @@
#include <atlctl.h>
// Copied min/max defs from windows headers to appease atlimage.h.
-// TODO(slightlyoff): Figure out of more recent platform SDK's (> 6.1)
+// TODO(slightlyoff): Figure out of more recent platform SDK's (> 6.1)
// undo the janky "#define NOMINMAX" train wreck. See:
// http://connect.microsoft.com/VisualStudio/feedback/ViewFeedback.aspx?FeedbackID=100703
#ifndef max
@@ -282,13 +282,13 @@ END_MSG_MAP()
return false;
}
- // Should connections initiated by this class try to block
- // responses served with the X-Frame-Options header?
- // ActiveX controls genereally will want to do this,
- // returning true, while true top-level documents
- // (ActiveDocument servers) will not. Your specialization
- // of this template should implement this method based on how
- // it "feels" from a security perspective. If it's hosted in another
+ // Should connections initiated by this class try to block
+ // responses served with the X-Frame-Options header?
+ // ActiveX controls genereally will want to do this,
+ // returning true, while true top-level documents
+ // (ActiveDocument servers) will not. Your specialization
+ // of this template should implement this method based on how
+ // it "feels" from a security perspective. If it's hosted in another
// scriptable document, return true, else false.
virtual bool is_frame_busting_enabled() const {
return true;
@@ -509,7 +509,7 @@ END_MSG_MAP()
// We can initiate navigation here even if ready_state is not complete.
// We do not have to set proxy, and AutomationClient will take care
// of navigation just after CreateExternalTab is done.
- if (!automation_client_->InitiateNavigation(full_url)) {
+ if (!automation_client_->InitiateNavigation(full_url, is_privileged_)) {
// TODO(robertshield): Make InitiateNavigation return more useful
// error information.
return E_INVALIDARG;