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

Unified Diff: cc/resources/video_resource_updater.cc

Issue 16136005: Update remaining files to use WeakPtr<T>::get() instead of "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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 | « no previous file | device/bluetooth/bluetooth_adapter_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/video_resource_updater.cc
diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc
index 45f86b06a28e39de9c713b529661aa571c347dac..a0012783a049f597f207c84d8de5f177202f9c54 100644
--- a/cc/resources/video_resource_updater.cc
+++ b/cc/resources/video_resource_updater.cc
@@ -365,7 +365,7 @@ void VideoResourceUpdater::ReturnTexture(
unsigned resource_id,
unsigned sync_point,
bool lost_resource) {
- if (!updater) {
+ if (!updater.get()) {
danakj 2013/06/04 14:01:14 Have you considered adding an operator bool() to r
akalin 2013/06/04 15:45:15 Yes, that will be done in a future CL, and a secon
danakj 2013/06/04 17:04:52 Great, thanks!
// Resource was already deleted.
return;
}
@@ -379,7 +379,7 @@ void VideoResourceUpdater::RecycleResource(
RecycleResourceData data,
unsigned sync_point,
bool lost_resource) {
- if (!updater) {
+ if (!updater.get()) {
// Resource was already deleted.
return;
}
« no previous file with comments | « no previous file | device/bluetooth/bluetooth_adapter_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698