Comment #3 on issue 28800 by [email protected]: PDF doesn't open http://code.google.com/p/chromium/issues/detail?id=28800
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=33673
------------------------------------------------------------------------
r33673 | [email protected] | 2009-12-02 23:57:44 -0800 (Wed, 02 Dec 2009)
| 13 lines
Changed paths:
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/npapi/plugin_ref_target_page.html
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/npapi/plugin_ref_target_page.html.mock-http-headers
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/npapi/plugin_url_request_referrer_test.html
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/ui/npapi_uitest.cc?r1=33673&r2=33672
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_client.cc?r1=33673&r2=33672
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_geturl_test.cc?r1=33673&r2=33672
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_geturl_test.h?r1=33673&r2=33672
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webplugin_impl.cc?r1=33673&r2=33672
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webplugin_impl.h?r1=33673&r2=33672
Don't set referrers on outgoing plugin requests if the load_manually flag
is set. This emulates the
behavior of other browsers and fixes
http://code.google.com/p/chromium/issues/detail?id=28800
I added a UI test to validate that the plugin source URL is set on outgoing
GetURL requests issued
by the plugin. To validate that the document URL is set as the referrer on
the initial URL request
would take some more work. Will try and add that in a future CL.
I also changed the WebPluginImpl::RouteToFrame function to set the referrer
on similar lines.
Bug=28800
Test=Covered by UI test
Review URL: http://codereview.chromium.org/459003
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
