<webview>: Fix resize.

content/browser/browser_plugin/browser_plugin_guest.cc

Index: content/browser/browser_plugin/browser_plugin_guest.cc
diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc
index 6dbf2df9fb5014bd680ff9792724601275cded24..e5fbfb1266335e095c0aa122d9f5b30f79c19e2a 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.cc
+++ b/content/browser/browser_plugin/browser_plugin_guest.cc
@@ -1249,6 +1249,8 @@ void BrowserPluginGuest::OnPluginDestroyed(int instance_id) {
 void BrowserPluginGuest::OnResizeGuest(
     int instance_id,
     const BrowserPluginHostMsg_ResizeGuest_Params& params) {
+  if (!params.size_changed)

[Cris Neckar, 2013/07/22 19:56:05]

I don't understand why you need to send this? If the message does nothing when
it is set couldn't you just as easily not send to message at all in this case?

[Fady Samuel, 2013/07/22 20:20:47]

This method is called from multiple places (not just
BrowserPluginHostMsg_ResizeGuest). The params are piggybacked off multiple
message types. Sometimes the fields of BrowserPluginHostMsg_ResizeGuest_Params
are not populated.

+    return;
   // BrowserPlugin manages resize flow control itself and does not depend
   // on RenderWidgetHost's mechanisms for flow control, so we reset those flags
   // here. If we are setting the size for the first time before navigating then
@@ -1479,7 +1481,6 @@ void BrowserPluginGuest::RequestMediaAccessPermission(
 
 void BrowserPluginGuest::OnUpdateRect(
     const ViewHostMsg_UpdateRect_Params& params) {
-
   BrowserPluginMsg_UpdateRect_Params relay_params;
   relay_params.view_size = params.view_size;
   relay_params.scale_factor = params.scale_factor;