Use either 0.1 or 1.0 versions of PPBFlashFullscreen.

ppapi/cpp/private/flash_fullscreen.cc

 // Copyright (c) 2011 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/cpp/private/flash_fullscreen.h"
 
 #include "ppapi/c/private/ppb_flash_fullscreen.h"
 #include "ppapi/cpp/instance.h"
 #include "ppapi/cpp/module.h"
 #include "ppapi/cpp/module_impl.h"
@@ skipping 14 matching lines @@
 }  // namespace
 
 FlashFullscreen::FlashFullscreen(const InstanceHandle& instance)
     : instance_(instance) {
 }
 
 FlashFullscreen::~FlashFullscreen() {
 }
 
 bool FlashFullscreen::IsFullscreen() {
-  return has_interface<PPB_FlashFullscreen_0_1>() &&
-      get_interface<PPB_FlashFullscreen_0_1>()->IsFullscreen(
-          instance_.pp_instance());
+  if (has_interface<PPB_FlashFullscreen_1_0>())
+    return get_interface<PPB_FlashFullscreen_1_0>()->IsFullscreen(

[viettrungluu, 2012/09/24 23:06:25]

Nit: since this spans > 1 line, you're supposed to use braces.

+        instance_.pp_instance());
+  if (has_interface<PPB_FlashFullscreen_0_1>())
+    return get_interface<PPB_FlashFullscreen_0_1>()->IsFullscreen(

[viettrungluu, 2012/09/24 23:06:25]

"

etc.

+        instance_.pp_instance());
+  return false;
 }
 
 bool FlashFullscreen::SetFullscreen(bool fullscreen) {
-  if (!has_interface<PPB_FlashFullscreen_0_1>())
-    return false;
-  return PP_ToBool(get_interface<PPB_FlashFullscreen_0_1>()->SetFullscreen(
-      instance_.pp_instance(), PP_FromBool(fullscreen)));
+  if (has_interface<PPB_FlashFullscreen_1_0>())
+    return PP_ToBool(get_interface<PPB_FlashFullscreen_1_0>()->SetFullscreen(
+        instance_.pp_instance(), PP_FromBool(fullscreen)));
+  if (has_interface<PPB_FlashFullscreen_0_1>())
+    return PP_ToBool(get_interface<PPB_FlashFullscreen_0_1>()->SetFullscreen(
+        instance_.pp_instance(), PP_FromBool(fullscreen)));
+  return false;
 }
 
 bool FlashFullscreen::GetScreenSize(Size* size) {
-  if (!has_interface<PPB_FlashFullscreen_0_1>())
-    return false;
-  return PP_ToBool(get_interface<PPB_FlashFullscreen_0_1>()->GetScreenSize(
-      instance_.pp_instance(), &size->pp_size()));
+  if (has_interface<PPB_FlashFullscreen_1_0>())
+    return PP_ToBool(get_interface<PPB_FlashFullscreen_1_0>()->GetScreenSize(
+        instance_.pp_instance(), &size->pp_size()));
+  if (has_interface<PPB_FlashFullscreen_0_1>())
+    return PP_ToBool(get_interface<PPB_FlashFullscreen_0_1>()->GetScreenSize(
+        instance_.pp_instance(), &size->pp_size()));
+  return false;
 }
 
 bool FlashFullscreen::MustRecreateContexts() {
   return !get_interface<PPB_FlashFullscreen_1_0>();
 }
 
 }  // namespace pp