diff options
author | neb@chromium.org <neb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-20 21:01:10 +0000 |
---|---|---|
committer | neb@chromium.org <neb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-20 21:01:10 +0000 |
commit | 6205759e4ec4bf1d9e85049cc1cc90d76c916283 (patch) | |
tree | 6fd797237c6f8ee0bc532747af2e004b2ca3a865 /webkit/glue/plugins/pepper_url_request_info.cc | |
parent | 92a9c68ab835fd2148a8cce64db74dd20e01e9dd (diff) | |
download | chromium_src-6205759e4ec4bf1d9e85049cc1cc90d76c916283.zip chromium_src-6205759e4ec4bf1d9e85049cc1cc90d76c916283.tar.gz chromium_src-6205759e4ec4bf1d9e85049cc1cc90d76c916283.tar.bz2 |
De-coupled resource references by plugin from Resource object's refcount. Made so that ResourceTracker keeps the refs-by-plugin count, and when the ppapi plugin releases the last reference, it UnRefs the resource.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2871027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53098 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/plugins/pepper_url_request_info.cc')
-rw-r--r-- | webkit/glue/plugins/pepper_url_request_info.cc | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/webkit/glue/plugins/pepper_url_request_info.cc b/webkit/glue/plugins/pepper_url_request_info.cc index 9ce79ce..d230f20 100644 --- a/webkit/glue/plugins/pepper_url_request_info.cc +++ b/webkit/glue/plugins/pepper_url_request_info.cc @@ -53,13 +53,12 @@ PP_Resource Create(PP_Module module_id) { return 0; URLRequestInfo* request = new URLRequestInfo(module); - request->AddRef(); // AddRef for the caller. - return request->GetResource(); + return request->GetReference(); } bool IsURLRequestInfo(PP_Resource resource) { - return !!Resource::GetAs<URLRequestInfo>(resource).get(); + return !!Resource::GetAs<URLRequestInfo>(resource); } bool SetProperty(PP_Resource request_id, @@ -67,7 +66,7 @@ bool SetProperty(PP_Resource request_id, PP_Var var) { scoped_refptr<URLRequestInfo> request( Resource::GetAs<URLRequestInfo>(request_id)); - if (!request.get()) + if (!request) return false; if (var.type == PP_VARTYPE_BOOL) @@ -82,7 +81,7 @@ bool SetProperty(PP_Resource request_id, bool AppendDataToBody(PP_Resource request_id, PP_Var var) { scoped_refptr<URLRequestInfo> request( Resource::GetAs<URLRequestInfo>(request_id)); - if (!request.get()) + if (!request) return false; String* data = GetString(var); @@ -99,11 +98,11 @@ bool AppendFileToBody(PP_Resource request_id, PP_Time expected_last_modified_time) { scoped_refptr<URLRequestInfo> request( Resource::GetAs<URLRequestInfo>(request_id)); - if (!request.get()) + if (!request) return false; scoped_refptr<FileRef> file_ref(Resource::GetAs<FileRef>(file_ref_id)); - if (!file_ref.get()) + if (!file_ref) return false; return request->AppendFileToBody(file_ref, |