Don't require a recreation of the context when fullscreening

ppapi/cpp/private/flash_fullscreen.cc

Index: ppapi/cpp/private/flash_fullscreen.cc
diff --git a/ppapi/cpp/private/flash_fullscreen.cc b/ppapi/cpp/private/flash_fullscreen.cc
index 8a5be263f64c754a588a6406e439cd7d7c6f08bc..dd17b1a15d452cf174ed769cee4a463607c1d347 100644
--- a/ppapi/cpp/private/flash_fullscreen.cc
+++ b/ppapi/cpp/private/flash_fullscreen.cc
@@ -14,8 +14,12 @@ 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
@@ -28,23 +32,27 @@ 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