Track whether a page is fetched via a proxy.
Created attachment 58204 [details] Simple patch - chromium specific.
Not sure if it matters but these fields will not be copied across threads (see PassOwnPtr<CrossThreadResourceRequestData> copyData() const;). This is called when workers do resource requests. Other: The reviewer typically isn't filled in until someone has r+'ed it.
Yes, CrossThreadResourceRequestData can't deal with platform specific flags. So this is no change. re the Reviewer: I know. I take a crap-shoot at the reviewer. If Dimitry reviews it, then I don't need to regenerate a patch. A better fix would be to get rid of ChangeLogs, but I think that is an issue for the Office of Redundancy to tackle...
(In reply to comment #3) > Yes, CrossThreadResourceRequestData can't deal with platform specific flags. So this is no change. Well, it doesn't at present. I suppose it should be modified to handle platform specific flags if they matter (when it is called). Specifically, this method is called when workers do resource requests. Do these fields matter for that case?
(In reply to comment #4) > Specifically, this method is called when workers do resource requests. Do these fields matter for that case? We're trying to track at a page-load-level whether a page is "proxied or not". This already has room for error, because some resources on a page could be proxied, and others may not. We use the base-page as the trigger for how to label the page load. Since we're only looking at page loads, not resources loaded by XHR, workers, etc, it doesn't matter for workers.
(In reply to comment #5) > (In reply to comment #4) > > Specifically, this method is called when workers do resource requests. Do these fields matter for that case? > > We're trying to track at a page-load-level whether a page is "proxied or not". This already has room for error, because some resources on a page could be proxied, and others may not. We use the base-page as the trigger for how to label the page load. Since we're only looking at page loads, not resources loaded by XHR, workers, etc, it doesn't matter for workers. Thanks!
Created attachment 58269 [details] Update the reviewer field .
Created attachment 58271 [details] One more try, without tagalong fixes.
Comment on attachment 58271 [details] One more try, without tagalong fixes. Clearing flags on attachment: 58271 Committed r60955: <http://trac.webkit.org/changeset/60955>
*** Bug 43011 has been marked as a duplicate of this bug. ***
This patch claims to have been landed.