Reimplement the Pepper fullscreen API to use webkitRequestFullScreen and friends.

content/renderer/pepper_plugin_delegate_impl.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 "content/renderer/pepper_plugin_delegate_impl.h"
 
 #include <cmath>
 #include <queue>
 
 #include "base/bind.h"
@@ skipping 1280 matching lines @@
   pending_connect_tcps_.Remove(request_id);
 
   connector->CompleteConnectTcp(socket, local_addr, remote_addr);
 }
 
 int32_t PepperPluginDelegateImpl::ShowContextMenu(
     webkit::ppapi::PluginInstance* instance,
     webkit::ppapi::PPB_Flash_Menu_Impl* menu,
     const gfx::Point& position) {
   int32 render_widget_id = render_view_->routing_id();
-  if (instance->IsFullscreen(instance->pp_instance())) {
-    webkit::ppapi::FullscreenContainer* container =
-        instance->fullscreen_container();
-    DCHECK(container);
-    render_widget_id =
-        static_cast<RenderWidgetFullscreenPepper*>(container)->routing_id();
-  }
 
   int request_id = pending_context_menus_.Add(
       new scoped_refptr<webkit::ppapi::PPB_Flash_Menu_Impl>(menu));
 
   ContextMenuParams params;
   params.x = position.x();
   params.y = position.y();
   params.custom_context.is_pepper_menu = true;
   params.custom_context.request_id = request_id;
   params.custom_context.render_widget_id = render_widget_id;
@@ skipping 43 matching lines @@
 
 void PepperPluginDelegateImpl::OnCustomContextMenuAction(
     const webkit_glue::CustomContextMenuContext& custom_context,
     unsigned action) {
   // Just save the action.
   DCHECK(!has_saved_context_menu_action_);
   has_saved_context_menu_action_ = true;
   saved_context_menu_action_ = action;
 }
 
-webkit::ppapi::FullscreenContainer*
-PepperPluginDelegateImpl::CreateFullscreenContainer(
-    webkit::ppapi::PluginInstance* instance) {
-  return render_view_->CreatePepperFullscreenContainer(instance);
+void PepperPluginDelegateImpl::SetFullscreen(

[brettw, 2011/08/24 23:14:04]

Do we even need this function anymore? The code in webkit/plugins/ppapi can just
do this call directly since it has access to the container().

[jeremya, 2011/08/24 23:40:45]

Done. Good catch :)

+    webkit::ppapi::PluginInstance* instance,
+    bool fullscreen) {
+  instance->container()->setFullscreen(fullscreen);
 }
 
 gfx::Size PepperPluginDelegateImpl::GetScreenSize() {
   WebKit::WebScreenInfo info = render_view_->screenInfo();
   return gfx::Size(info.rect.width, info.rect.height);
 }
 
 std::string PepperPluginDelegateImpl::GetDefaultEncoding() {
   // TODO(brettw) bug 56615: Somehow get the preference for the default
   // encoding here rather than using the global default for the UI language.
@@ skipping 90 matching lines @@
 
 ppapi::Preferences PepperPluginDelegateImpl::GetPreferences() {
   return ppapi::Preferences(render_view_->webkit_preferences());
 }
 
 void PepperPluginDelegateImpl::PublishInitialPolicy(
     scoped_refptr<webkit::ppapi::PluginInstance> instance,
     const std::string& policy) {
   instance->HandlePolicyUpdate(policy);
 }