Don't require a recreation of the context when fullscreening

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"
 #include "ppapi/cpp/size.h"
 
 namespace pp {
 
 namespace {
 
-template <> const char* interface_name<PPB_FlashFullscreen>() {
-  return PPB_FLASHFULLSCREEN_INTERFACE;
+template <> const char* interface_name<PPB_FlashFullscreen_0_1>() {
+  return PPB_FLASHFULLSCREEN_INTERFACE_0_1;
+}
+
+template <> const char* interface_name<PPB_FlashFullscreen_1_0>() {
+  return PPB_FLASHFULLSCREEN_INTERFACE_1_0;
 }
 
 }  // namespace
 
 FlashFullscreen::FlashFullscreen(const InstanceHandle& instance)
     : instance_(instance) {
 }
 
 FlashFullscreen::~FlashFullscreen() {
 }
 
 bool FlashFullscreen::IsFullscreen() {
-  return has_interface<PPB_FlashFullscreen>() &&
-      get_interface<PPB_FlashFullscreen>()->IsFullscreen(
+  return has_interface<PPB_FlashFullscreen_0_1>() &&

[viettrungluu, 2012/09/20 21:13:51]

You should really do:

if (has_interface<PPB_FlashFullscreen_1_0>())
  return get_interface<..._1_0>...;
if (has_interface<PPB_FlashFullscreen_0_1>())
  return get_interface<..._0_1>...;
return false;

This makes it possible to actually remove support for 0.1 (in Chrome) at some
point. It also makes it easy to remove the corresponding support in the wrapper.

+      get_interface<PPB_FlashFullscreen_0_1>()->IsFullscreen(
           instance_.pp_instance());
 }
 
 bool FlashFullscreen::SetFullscreen(bool fullscreen) {
-  if (!has_interface<PPB_FlashFullscreen>())
+  if (!has_interface<PPB_FlashFullscreen_0_1>())

[viettrungluu, 2012/09/20 21:13:51]

"

     return false;
-  return PP_ToBool(get_interface<PPB_FlashFullscreen>()->SetFullscreen(
+  return PP_ToBool(get_interface<PPB_FlashFullscreen_0_1>()->SetFullscreen(
       instance_.pp_instance(), PP_FromBool(fullscreen)));
 }
 
 bool FlashFullscreen::GetScreenSize(Size* size) {
-  if (!has_interface<PPB_FlashFullscreen>())
+  if (!has_interface<PPB_FlashFullscreen_0_1>())

[viettrungluu, 2012/09/20 21:13:51]

"

     return false;
-  return PP_ToBool(get_interface<PPB_FlashFullscreen>()->GetScreenSize(
+  return PP_ToBool(get_interface<PPB_FlashFullscreen_0_1>()->GetScreenSize(
       instance_.pp_instance(), &size->pp_size()));
 }
 
+bool FlashFullscreen::MustRecreateContexts() {
+  return !get_interface<PPB_FlashFullscreen_1_0>();
+}
+
 }  // namespace pp