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

Unified Diff: ppapi/proxy/ppb_video_capture_proxy.cc

Issue 8342016: Revert 106142 - Add a new globals object for PPAPI tracking information. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 2 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
« no previous file with comments | « ppapi/proxy/ppb_var_proxy.cc ('k') | ppapi/proxy/ppp_instance_private_proxy_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/ppb_video_capture_proxy.cc
===================================================================
--- ppapi/proxy/ppb_video_capture_proxy.cc (revision 106146)
+++ ppapi/proxy/ppb_video_capture_proxy.cc (working copy)
@@ -373,11 +373,11 @@
const PP_VideoCaptureDeviceInfo_Dev& info,
const std::vector<PPPVideoCapture_Buffer>& buffers) {
EnterPluginFromHostResource<PPB_VideoCapture_API> enter(host_resource);
- if (enter.failed() || !ppp_video_capture_impl_)
+ PluginResourceTracker* tracker = PluginResourceTracker::GetInstance();
+ PP_Resource resource = tracker->PluginResourceForHostResource(host_resource);
+ if (!resource || !ppp_video_capture_impl_ || enter.failed())
return;
- PluginResourceTracker* tracker =
- PluginGlobals::Get()->plugin_resource_tracker();
scoped_array<PP_Resource> resources(new PP_Resource[buffers.size()]);
for (size_t i = 0; i < buffers.size(); ++i) {
// We assume that the browser created a new set of resources.
@@ -386,12 +386,10 @@
buffers[i].handle,
buffers[i].size);
}
-
- VideoCapture* capture = static_cast<VideoCapture*>(enter.object());
- capture->SetBufferCount(buffers.size());
+ static_cast<VideoCapture*>(enter.object())->SetBufferCount(buffers.size());
ppp_video_capture_impl_->OnDeviceInfo(
host_resource.instance(),
- capture->pp_resource(),
+ resource,
&info,
buffers.size(),
resources.get());
@@ -402,38 +400,40 @@
void PPP_VideoCapture_Proxy::OnMsgOnStatus(const HostResource& host_resource,
uint32_t status) {
EnterPluginFromHostResource<PPB_VideoCapture_API> enter(host_resource);
- if (enter.failed() || !ppp_video_capture_impl_)
+ PluginResourceTracker* tracker = PluginResourceTracker::GetInstance();
+ PP_Resource resource = tracker->PluginResourceForHostResource(host_resource);
+ if (!resource || !ppp_video_capture_impl_ || enter.failed())
return;
- VideoCapture* capture = static_cast<VideoCapture*>(enter.object());
- if (!capture->OnStatus(status))
+ if (!static_cast<VideoCapture*>(enter.object())->OnStatus(status))
return;
ppp_video_capture_impl_->OnStatus(
- host_resource.instance(), capture->pp_resource(), status);
+ host_resource.instance(), resource, status);
}
void PPP_VideoCapture_Proxy::OnMsgOnError(const HostResource& host_resource,
uint32_t error_code) {
EnterPluginFromHostResource<PPB_VideoCapture_API> enter(host_resource);
- if (enter.failed() || !ppp_video_capture_impl_)
+ PluginResourceTracker* tracker = PluginResourceTracker::GetInstance();
+ PP_Resource resource = tracker->PluginResourceForHostResource(host_resource);
+ if (!resource || !ppp_video_capture_impl_ || enter.failed())
return;
-
- VideoCapture* capture = static_cast<VideoCapture*>(enter.object());
- capture->set_status(PP_VIDEO_CAPTURE_STATUS_STOPPED);
+ static_cast<VideoCapture*>(enter.object())->set_status(
+ PP_VIDEO_CAPTURE_STATUS_STOPPED);
ppp_video_capture_impl_->OnError(
- host_resource.instance(), capture->pp_resource(), error_code);
+ host_resource.instance(), resource, error_code);
}
void PPP_VideoCapture_Proxy::OnMsgOnBufferReady(
const HostResource& host_resource, uint32_t buffer) {
EnterPluginFromHostResource<PPB_VideoCapture_API> enter(host_resource);
- if (enter.failed() || !ppp_video_capture_impl_)
+ PluginResourceTracker* tracker = PluginResourceTracker::GetInstance();
+ PP_Resource resource = tracker->PluginResourceForHostResource(host_resource);
+ if (!resource || !ppp_video_capture_impl_ || enter.failed())
return;
-
- VideoCapture* capture = static_cast<VideoCapture*>(enter.object());
- capture->SetBufferInUse(buffer);
+ static_cast<VideoCapture*>(enter.object())->SetBufferInUse(buffer);
ppp_video_capture_impl_->OnBufferReady(
- host_resource.instance(), capture->pp_resource(), buffer);
+ host_resource.instance(), resource, buffer);
}
} // namespace proxy
« no previous file with comments | « ppapi/proxy/ppb_var_proxy.cc ('k') | ppapi/proxy/ppp_instance_private_proxy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698