WebWidgetClient::screenInfo() no longer does a synchronous IPC.

content/renderer/render_widget.h

 // 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.
 
 #ifndef CONTENT_RENDERER_RENDER_WIDGET_H_
 #define CONTENT_RENDERER_RENDER_WIDGET_H_
 #pragma once
 
 #include <deque>
 #include <vector>
@@ skipping 58 matching lines @@
 // a RenderWidgetHost, the latter of which lives in a different process.
 class CONTENT_EXPORT RenderWidget
     : public IPC::Channel::Listener,
       public IPC::Message::Sender,
       NON_EXPORTED_BASE(virtual public WebKit::WebWidgetClient),
       public base::RefCounted<RenderWidget> {
  public:
   // Creates a new RenderWidget.  The opener_id is the routing ID of the
   // RenderView that this widget lives inside.
   static RenderWidget* Create(int32 opener_id,
-                              WebKit::WebPopupType popup_type);
+                              WebKit::WebPopupType popup_type,
+                              const WebKit::WebScreenInfo& screen_info);
 
   // Creates a WebWidget based on the popup type.
   static WebKit::WebWidget* CreateWebWidget(RenderWidget* render_widget);
 
   // The compositing surface assigned by the RenderWidgetHost
   // (or RenderViewHost). Will be gfx::kNullPluginWindow if not assigned yet,
   // in which case we should not create any GPU command buffers with it.
   // The routing ID assigned by the RenderProcess. Will be MSG_ROUTING_NONE if
   // not yet assigned a view ID, in which case, the process MUST NOT send
   // messages with this ID to the parent.
@@ skipping 60 matching lines @@
     return filtered_time_per_frame_;
   }
 
  protected:
   // Friend RefCounted so that the dtor can be non-public. Using this class
   // without ref-counting is an error.
   friend class base::RefCounted<RenderWidget>;
   // For unit tests.
   friend class RenderWidgetTest;
 
-  explicit RenderWidget(WebKit::WebPopupType popup_type);
+  explicit RenderWidget(WebKit::WebPopupType popup_type,

[jam, 2012/02/29 23:41:17]

nit: no need for explicit

[Fady Samuel, 2012/03/01 00:52:25]

True, thanks.

+                        const WebKit::WebScreenInfo& screen_info);
   virtual ~RenderWidget();
 
   // Initializes this view with the given opener.  CompleteInit must be called
   // later.
   void Init(int32 opener_id);
 
   // Called by Init and subclasses to perform initialization.
   void DoInit(int32 opener_id,
               WebKit::WebWidget* web_widget,
               IPC::SyncMessage* create_widget_message);
@@ skipping 318 matching lines @@
   // pass state between DoDeferredUpdate and OnSwapBuffersPosted.
   scoped_ptr<ViewHostMsg_UpdateRect_Params> pending_update_params_;
 
   // Queue of UpdateRect messages corresponding to a SwapBuffers. We want to
   // delay sending of UpdateRect until the corresponding SwapBuffers has been
   // executed. Since we can have several in flight, we need to keep them in a
   // queue. Note: some SwapBuffers may not correspond to an update, in which
   // case NULL is added to the queue.
   std::deque<ViewHostMsg_UpdateRect*> updates_pending_swap_;
 
+  // Properties of the screen hosting this RenderWidget instance.
+  WebKit::WebScreenInfo screen_info_;
+
   // Set to true if we should invert all pixels.
   bool invert_;
 
   // The Skia paint object for inverting.
   scoped_ptr<SkPaint> invert_paint_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidget);
 };
 
 #endif  // CONTENT_RENDERER_RENDER_WIDGET_H_