cc: Move video upload to VideoResourceUpdater.

cc/resources/resource_provider.h

Index: cc/resources/resource_provider.h
diff --git a/cc/resources/resource_provider.h b/cc/resources/resource_provider.h
index 56daa3e534f5eb37fc009e88a91a9b3d061ef903..641b2ff2c1d68fe69684fa2555d9480404d764e5 100644
--- a/cc/resources/resource_provider.h
+++ b/cc/resources/resource_provider.h
@@ -14,6 +14,7 @@
 #include "base/callback.h"
 #include "base/hash_tables.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
 #include "base/threading/thread_checker.h"
 #include "cc/base/cc_export.h"
 #include "cc/output/context_provider.h"
@@ -39,8 +40,11 @@ namespace cc {
 class TextureUploader;
 
 // This class is not thread-safe and can only be called from the thread it was
-// created on (in practice, the impl thread).
-class CC_EXPORT ResourceProvider {
+// created on (in practice, the impl thread). It supports WeakPtr in order to
+// tell when the provider and the context it holds is destroyed and no longer
+// usable.
+class CC_EXPORT ResourceProvider
+    : public base::SupportsWeakPtr<ResourceProvider> {

[piman, 2013/04/08 21:03:47]

Will the VideoResourceUpdater outlive the layer? It doesn't now but I'm not
entirely sure where this is going.
If it doesn't outlive the layer, then we can avoid this by relying on the
DidLoseOutputSurface callback in the layer.

I'm somewhat worried that serving a weak pointer here gives an easy way out, but
may prove problematic for cases where the RP gets destroyed without a context
loss (e.g. compositor teardown), but before the resources are properly released
- and it's then too late and we leak them.

[danakj, 2013/04/08 21:44:56]

Removed this as per offline convo. The callback is called from RP, so it doesn't
need to worry about getting called after the RP is destroyed.

  public:
   typedef unsigned ResourceId;
   typedef std::vector<ResourceId> ResourceIdArray;