Fix Sunspider performance regression

content/browser/browser_plugin/browser_plugin_embedder.cc

 // Copyright (c) 2012 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/browser/browser_plugin/browser_plugin_embedder.h"
 
 #include "base/stl_util.h"
 #include "content/browser/browser_plugin/browser_plugin_guest.h"
 #include "content/browser/browser_plugin/browser_plugin_host_factory.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
@@ skipping 118 matching lines @@
       guest_web_contents->GetMutableRendererPrefs();
   // Copy renderer preferences (and nothing else) from the embedder's
   // TabContents to the guest.
   //
   // For GTK and Aura this is necessary to get proper renderer configuration
   // values for caret blinking interval, colors related to selection and
   // focus.
   *guest_renderer_prefs = *web_contents()->GetMutableRendererPrefs();
 
   guest_renderer_prefs->throttle_input_events = false;
+  // We would like the guest to report changes to frame names so that we can
+  // update the BrowserPlugin's corresponding 'name' attribute.

[Charlie Reis, 2013/01/09 22:35:06]

Please add a TODO with a bug number to remove this.  This isn't a long-term fix,
because it slows down SunSpider-like workloads (whether this many name changes
is realistic in practice or not) within the webview tag, and it means we can't
keep track of window names in normal pages.

[Fady Samuel, 2013/01/09 22:40:57]

Done.

+  guest_renderer_prefs->report_frame_name_changes = true;
   // Navigation is disabled in Chrome Apps. We want to make sure guest-initiated
   // navigations still continue to function inside the app.
   guest_renderer_prefs->browser_handles_all_top_level_requests = false;
   AddGuest(instance_id, guest_web_contents);
   guest_web_contents->SetDelegate(guest);
 
   // Create a swapped out RenderView for the guest in the embedder render
   // process, so that the embedder can access the guest's window object.
   int guest_routing_id =
       static_cast<WebContentsImpl*>(guest->GetWebContents())->
@@ skipping 202 matching lines @@
                                              uint32 sync_point) {
   AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
   ack_params.mailbox_name = mailbox_name;
   ack_params.sync_point = sync_point;
   RenderWidgetHostImpl::AcknowledgeBufferPresent(route_id,
                                                  gpu_host_id,
                                                  ack_params);
 }
 
 }  // namespace content