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

Unified Diff: webkit/glue/plugins/pepper_resource_tracker.cc

Issue 5685002: Make it possible to write simple unit tests for the pepper implementation. Th... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years 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
Index: webkit/glue/plugins/pepper_resource_tracker.cc
===================================================================
--- webkit/glue/plugins/pepper_resource_tracker.cc (revision 68719)
+++ webkit/glue/plugins/pepper_resource_tracker.cc (working copy)
@@ -22,6 +22,9 @@
return result->second.first;
}
+// static
+ResourceTracker* ResourceTracker::singleton_override_ = NULL;
+
ResourceTracker::ResourceTracker()
: last_id_(0) {
}
@@ -31,6 +34,8 @@
// static
ResourceTracker* ResourceTracker::Get() {
+ if (singleton_override_)
+ return singleton_override_;
return Singleton<ResourceTracker>::get();
}
@@ -73,7 +78,7 @@
void ResourceTracker::ForceDeletePluginResourceRefs(PP_Resource res) {
ResourceMap::iterator i = live_resources_.find(res);
- if (i != live_resources_.end())
+ if (i == live_resources_.end())
return; // Nothing to do.
i->second.second = 0;
@@ -169,4 +174,16 @@
return found->second;
}
+// static
+void ResourceTracker::SetSingletonOverride(ResourceTracker* tracker) {
+ DCHECK(!singleton_override_);
+ singleton_override_ = tracker;
+}
+
+// static
+void ResourceTracker::ClearSingletonOverride() {
+ DCHECK(singleton_override_);
+ singleton_override_ = NULL;
+}
+
} // namespace pepper

Powered by Google App Engine
This is Rietveld 408576698