Chromium Code Reviews| Index: chromecast/base/cast_resource.h |
| diff --git a/chromecast/base/cast_resource.h b/chromecast/base/cast_resource.h |
| index cd248d77edc26872e426c5c2a42ef54a2aac1e0a..84c92f4423d16ec2bd3c2dcaf449e87d51114119 100644 |
| --- a/chromecast/base/cast_resource.h |
| +++ b/chromecast/base/cast_resource.h |
| @@ -53,6 +53,14 @@ class CastResource { |
| }; |
| void SetCastResourceClient(Client* client); |
| + // Called to initialize resources after OEM has granted them, and before |
| + // they start getting used by consumers. Implementation must be synchronous |
| + // since consumers will start using the resource immediately afterwards. |
| + // TODO(esum): We should allow this method to be asynchronous in case an |
| + // implementer needs to make expensive calls and doesn't want to block the |
| + // UI thread (b/26239576). For now, don't do anything expensive in your |
| + // implementation; if you really need to, then this bug has to be resolved. |
| + virtual void InitializeResource(Resource resource) = 0; |
|
byungchul
2015/12/18 00:53:21
GrantResource?
esum
2015/12/21 00:39:21
I actually prefer InitializeResource since this se
derekjchow1
2016/01/04 17:29:10
How about AcquireResource, since this is the oppos
esum
2016/01/11 16:28:57
Done. Had to rearrange some names to make things c
|
| // Called to release resources. Implementation should call |
| // Client::OnResourceReleased when resource is released on its side. |
| virtual void ReleaseResource(Resource resource) = 0; |