Pepper API implementation for output protection.

ppapi/proxy/output_protection_resource.h

+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef PPAPI_PROXY_OUTPUT_PROTECTION_RESOURCE_H_
+#define PPAPI_PROXY_OUTPUT_PROTECTION_RESOURCE_H_
+
+#include "base/compiler_specific.h"
+#include "ppapi/c/private/ppb_output_protection_private.h"
+#include "ppapi/proxy/device_enumeration_resource_helper.h"
+#include "ppapi/proxy/plugin_resource.h"
+#include "ppapi/thunk/ppb_output_protection_api.h"
+
+namespace ppapi {
+namespace proxy {
+
+class OutputProtectionResource
+    : public PluginResource,
+      public ::ppapi::thunk::PPB_OutputProtection_API {
+ public:
+  OutputProtectionResource(Connection connection,
+                           PP_Instance instance);
+  virtual ~OutputProtectionResource();

[dmichael (off chromium), 2013/09/09 20:37:15]

nit: the destructor and below can be private.

[kcwu, 2013/09/10 12:50:21]

Done.
It compiles but I don't understand.
This class want to implement PluginResource and PPB_OutputProtection_API's
public methods. Why make them private?

[dmichael (off chromium), 2013/09/11 18:22:45]

Not everyone in Chromium likes to do it this way, but it's come up a couple of
times in chromium-dev. E.g.:
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/CwzjcWd9cYM/TcUUl...
Basically, these are methods (including the destructor) that we expect by-design
to only be called through the interface. If somebody tries to call them
_directly_, I want the compiler to yell at them so they stop and think about
whether they're doing the right thing.

[kcwu, 2013/09/12 18:22:08]

Done.

+
+  // PluginResource overrides.
+  virtual thunk::PPB_OutputProtection_API* AsPPB_OutputProtection_API()
+      OVERRIDE;
+
+  // PPB_OutputProtection_API implementation.
+  virtual int32_t QueryStatus(uint32_t* link_mask, uint32_t* protection_mask,
+                              scoped_refptr<TrackedCallback> callback) OVERRIDE;
+  virtual int32_t EnableProtection(
+      uint32_t desired_method_mask,
+      scoped_refptr<TrackedCallback> callback) OVERRIDE;
+
+ private:
+  void OnPluginMsgQueryStatusReply(
+      const ResourceMessageReplyParams& params,
+      uint32_t link_mask,
+      uint32_t protection_mask);
+  void OnPluginMsgEnableProtectionReply(
+      const ResourceMessageReplyParams& params);
+
+  uint32_t* link_mask_;
+  uint32_t* protection_mask_;
+  scoped_refptr<TrackedCallback> query_status_callback_;
+  scoped_refptr<TrackedCallback> enable_protection_callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(OutputProtectionResource);
+};
+
+}  // namespace proxy
+}  // namespace ppapi
+
+#endif  // PPAPI_PROXY_OUTPUT_PROTECTION_RESOURCE_H_