Chromium Code Reviews| Index: chrome/browser/renderer_host/pepper/pepper_output_protection_message_filter.h |
| diff --git a/chrome/browser/renderer_host/pepper/pepper_output_protection_message_filter.h b/chrome/browser/renderer_host/pepper/pepper_output_protection_message_filter.h |
| index fe4865b8bb871664a1aeb5b12191237ac915fc55..cacd08247ab794e1a8e56be3fcf94218ab2bbe63 100644 |
| --- a/chrome/browser/renderer_host/pepper/pepper_output_protection_message_filter.h |
| +++ b/chrome/browser/renderer_host/pepper/pepper_output_protection_message_filter.h |
| @@ -23,14 +23,10 @@ struct HostMessageContext; |
| } // namespace host |
| } // namespace ppapi |
| -#if defined(OS_CHROMEOS) |
| -namespace chromeos { |
| -class OutputProtectionDelegate; |
| -} |
| -#endif |
| - |
| namespace chrome { |
| +class OutputProtectionProxy; |
| + |
| class PepperOutputProtectionMessageFilter |
| : public ppapi::host::ResourceMessageFilter { |
| public: |
| @@ -60,10 +56,9 @@ class PepperOutputProtectionMessageFilter |
| ppapi::host::ReplyMessageContext reply_context, |
| bool success); |
| -#if defined(OS_CHROMEOS) |
| - // Delegate. Should be deleted in UI thread. |
| - chromeos::OutputProtectionDelegate* delegate_; |
| -#endif |
| + // Owned by |this| and will be deleted on the UI thread. |
| + // TODO(xhwang): Use unique_ptr to avoid the confusion on ownership. |
|
ddorwin
2016/06/23 19:24:28
Is there a reason this can't be done now?
xhwang
2016/06/23 20:53:49
I wasn't super sure how to do this with unique_ptr
|
| + OutputProtectionProxy* proxy_; |
| base::WeakPtrFactory<PepperOutputProtectionMessageFilter> weak_ptr_factory_; |