Refactor PPB_Flash_Fullscreen to the new resource model.

ppapi/proxy/flash_fullscreen_resource.cc

Index: ppapi/proxy/flash_fullscreen_resource.cc
diff --git a/ppapi/proxy/flash_fullscreen_resource.cc b/ppapi/proxy/flash_fullscreen_resource.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c1022524c1d31797df3e63e20c1f0e8dea51a12a
--- /dev/null
+++ b/ppapi/proxy/flash_fullscreen_resource.cc
@@ -0,0 +1,47 @@
+// Copyright (c) 2012 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.
+
+#include "ppapi/proxy/flash_fullscreen_resource.h"
+
+#include "ppapi/c/pp_bool.h"
+#include "ppapi/proxy/plugin_dispatcher.h"

[dmichael (off chromium), 2012/11/27 20:58:01]

^^^ Do you need this?

[raymes, 2012/11/27 21:24:28]

Done.

+#include "ppapi/proxy/ppapi_messages.h"
+
+namespace ppapi {
+namespace proxy {
+
+FlashFullscreenResource::FlashFullscreenResource(Connection connection,
+                                                 PP_Instance instance)
+    : PluginResource(connection, instance),
+      is_fullscreen_(PP_FALSE) {
+}
+
+FlashFullscreenResource::~FlashFullscreenResource() {
+}
+
+thunk::PPB_Flash_Fullscreen_API*
+FlashFullscreenResource::AsPPB_Flash_Fullscreen_API() {
+  return this;
+}
+
+PP_Bool FlashFullscreenResource::IsFullscreen(PP_Instance instance) {
+  return is_fullscreen_;
+}
+
+PP_Bool FlashFullscreenResource::SetFullscreen(PP_Instance instance,
+                                               PP_Bool fullscreen) {
+  if (!sent_create_to_renderer())
+    SendCreate(RENDERER, PpapiHostMsg_FlashFullscreen_Create());
+  int32_t result = SyncCall<IPC::Message>(RENDERER,
+      PpapiHostMsg_FlashFullscreen_SetFullscreen(fullscreen));
+  return PP_FromBool(result == PP_OK);
+}
+
+void FlashFullscreenResource::SetFullscreenInternal(PP_Instance instance,

[dmichael (off chromium), 2012/11/27 20:58:01]

Maybe SetLocalIsFullscreen, or something? It's not very descriptive;
SetFullscreenInternal sounds almost like it's going to actually change the
fullscreen state.

[raymes, 2012/11/27 21:24:28]

You're right. I prefer that. I also improved the comment in the interface
definition.

+                                                    PP_Bool is_fullscreen) {
+  is_fullscreen_ = is_fullscreen;
+}
+
+}  // namespace proxy
+}  // namespace ppapi