diff options
author | tommi@chromium.org <tommi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-01 16:00:21 +0000 |
---|---|---|
committer | tommi@chromium.org <tommi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-01 16:00:21 +0000 |
commit | 10bf1a7030eeed02e2af95eaff012d2428ea5bc2 (patch) | |
tree | f381f0333bb9901aca6d2d5e2b0fdbfc5743b6d2 /chrome_frame/chrome_frame_activex_base.h | |
parent | 7f14e1fd31ddd510b19e2d9f6942c45435c9a8c6 (diff) | |
download | chromium_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.h | 18 |
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; |