Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(924)

Unified Diff: chrome/common/render_messages.h

Issue 46094: Fix our handling of mixed SSL / non-SSL content.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/render_messages.h
===================================================================
--- chrome/common/render_messages.h (revision 11701)
+++ chrome/common/render_messages.h (working copy)
@@ -242,6 +242,14 @@
// The referrer to use (may be empty).
GURL referrer;
+ // The origin of the frame that is associated with this request. This is used
+ // to update our mixed content state.
+ std::string frame_origin;
+
+ // The origin of the main frame (top-level frame) that is associated with this
+ // request. This is used to update our mixed content state.
+ std::string main_frame_origin;
+
// Additional HTTP request headers.
std::string headers;
@@ -255,10 +263,6 @@
// object).
ResourceType::Type resource_type;
- // True if this request is for a resource loaded over HTTP when the main page
- // was loaded over HTTPS.
- bool mixed_content;
-
// Used by plugin->browser requests to get the correct URLRequestContext.
uint32 request_context;
@@ -1244,11 +1248,12 @@
WriteParam(m, p.url);
WriteParam(m, p.policy_url);
WriteParam(m, p.referrer);
+ WriteParam(m, p.frame_origin);
+ WriteParam(m, p.main_frame_origin);
WriteParam(m, p.headers);
WriteParam(m, p.load_flags);
WriteParam(m, p.origin_pid);
WriteParam(m, p.resource_type);
- WriteParam(m, p.mixed_content);
WriteParam(m, p.request_context);
WriteParam(m, p.upload_content);
}
@@ -1258,11 +1263,12 @@
ReadParam(m, iter, &r->url) &&
ReadParam(m, iter, &r->policy_url) &&
ReadParam(m, iter, &r->referrer) &&
+ ReadParam(m, iter, &r->frame_origin) &&
+ ReadParam(m, iter, &r->main_frame_origin) &&
ReadParam(m, iter, &r->headers) &&
ReadParam(m, iter, &r->load_flags) &&
ReadParam(m, iter, &r->origin_pid) &&
ReadParam(m, iter, &r->resource_type) &&
- ReadParam(m, iter, &r->mixed_content) &&
ReadParam(m, iter, &r->request_context) &&
ReadParam(m, iter, &r->upload_content);
}
@@ -1274,14 +1280,16 @@
l->append(L", ");
LogParam(p.referrer, l);
l->append(L", ");
+ LogParam(p.frame_origin, l);
+ l->append(L", ");
+ LogParam(p.main_frame_origin, l);
+ l->append(L", ");
LogParam(p.load_flags, l);
l->append(L", ");
LogParam(p.origin_pid, l);
l->append(L", ");
LogParam(p.resource_type, l);
l->append(L", ");
- LogParam(p.mixed_content, l);
- l->append(L", ");
LogParam(p.request_context, l);
l->append(L")");
}

Powered by Google App Engine
This is Rietveld 408576698