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

Unified Diff: chrome_frame/plugin_url_request.h

Issue 386014: The ChromeFrameAutomationClient class needs to be refcounted as it implements... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month 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
« no previous file with comments | « chrome_frame/np_proxy_service.h ('k') | chrome_frame/test/chrome_frame_automation_mock.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/plugin_url_request.h
===================================================================
--- chrome_frame/plugin_url_request.h (revision 31733)
+++ chrome_frame/plugin_url_request.h (working copy)
@@ -21,7 +21,9 @@
// Interface for a class that keeps a collection of outstanding
// reqeusts and offers an outgoing channel.
-class PluginRequestHandler : public IPC::Message::Sender {
+class PluginRequestHandler
+ : public IPC::Message::Sender,
+ public base::RefCountedThreadSafe<PluginRequestHandler> {
public:
virtual bool AddRequest(PluginUrlRequest* request) = 0;
virtual void RemoveRequest(PluginUrlRequest* request) = 0;
@@ -127,7 +129,7 @@
bool frame_busting_enabled_;
private:
- PluginRequestHandler* request_handler_;
+ scoped_refptr<PluginRequestHandler> request_handler_;
int tab_;
int remote_request_id_;
uint64 post_data_len_;
« no previous file with comments | « chrome_frame/np_proxy_service.h ('k') | chrome_frame/test/chrome_frame_automation_mock.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698