use accelerated compositing in fullscreen pepper

webkit/plugins/ppapi/ppapi_plugin_instance.cc

 // Copyright (c) 2010 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 "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/scoped_ptr.h"
 #include "base/utf_string_conversions.h"
@@ skipping 384 matching lines @@
 }
 
 unsigned PluginInstance::GetBackingTextureId() {
   if (!bound_graphics_3d())
     return 0;
 
   return bound_graphics_3d()->GetBackingTextureId();
 }
 
 void PluginInstance::CommitBackingTexture() {
-  container_->commitBackingTexture();
+  if (fullscreen_container_) {
+    fullscreen_container_->Invalidate();
+  } else {
+    container_->commitBackingTexture();
+  }
 }
 
 PP_Var PluginInstance::GetWindowObject() {
   if (!container_)
     return PP_MakeUndefined();
 
   WebFrame* frame = container_->element().document().frame();
   if (!frame)
     return PP_MakeUndefined();
 
@@ skipping 513 matching lines @@
   WebURLRequest request(complete_url);
   document.frame()->setReferrerForRequest(request, GURL());
   request.setHTTPMethod(WebString::fromUTF8("GET"));
   request.setFirstPartyForCookies(document.firstPartyForCookies());
 
   WebString target_str = WebString::fromUTF8(target);
   container_->loadFrameRequest(request, target_str, false, NULL);
   return true;
 }
 
+PluginDelegate::PlatformContext3D* PluginInstance::CreateContext3D() {
+  if (fullscreen_container_) {

[brettw, 2011/01/06 17:52:32]

To match the surrounding style, don't use {} for single-line conditionals.

[piman, 2011/01/06 20:18:25]

It has an else clause - it's required in that case, isn't it ?

[brettw, 2011/01/06 20:22:13]

Nope, the only thing different about this is if one of the arms has it, all of
them must.

[piman, 2011/01/06 20:38:58]

Aaight, done.

+    return fullscreen_container_->CreateContext3D();
+  } else {
+    return delegate_->CreateContext3D();
+  }
+}
+
 bool PluginInstance::PrintPDFOutput(PP_Resource print_output,
                                     WebKit::WebCanvas* canvas) {
   scoped_refptr<PPB_Buffer_Impl> buffer(
       Resource::GetAs<PPB_Buffer_Impl>(print_output));
   if (!buffer.get() || !buffer->is_mapped() || !buffer->size()) {
     NOTREACHED();
     return false;
   }
 #if defined(OS_WIN)
   // For Windows, we need the PDF DLL to render the output PDF to a DC.
@@ skipping 224 matching lines @@
 PPB_Surface3D_Impl* PluginInstance::bound_graphics_3d() const {
   if (bound_graphics_.get() == NULL)
     return NULL;
 
   return bound_graphics_->Cast<PPB_Surface3D_Impl>();
 }
 
 }  // namespace ppapi
 }  // namespace webkit