Add pixel readback to page popup

public/web/WebWidget.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef WebWidget_h
 #define WebWidget_h
 
 #include "../platform/WebCanvas.h"
 #include "../platform/WebCommon.h"
+#include "../platform/WebPoint.h"
 #include "../platform/WebRect.h"
 #include "../platform/WebSize.h"
 #include "WebBeginFrameArgs.h"
 #include "WebCompositionUnderline.h"
 #include "WebTextDirection.h"
 #include "WebTextInputInfo.h"
 
 namespace blink {
 
 class WebCompositeAndReadbackAsyncCallback;
 class WebInputEvent;
 class WebLayerTreeView;
 class WebMouseEvent;
 class WebString;
 struct WebPoint;
 struct WebRenderingStats;
 template <typename T> class WebVector;
+class WebPagePopup;

[tkent, 2014/09/05 01:59:05]

nit: Please insert this between WebMouseEvent and WebString.

[keishi, 2014/09/11 12:53:26]

Done.

 
 class WebWidget {
 public:
     // This method closes and deletes the WebWidget.
     virtual void close() { }
 
     // Returns the current size of the WebWidget.
     virtual WebSize size() { return WebSize(); }
 
+    virtual WebPoint screenPosition() { return WebPoint(); }

[tkent, 2014/09/05 01:59:05]

This should not be necessary because we already have
WebWidgetClient::windowRect().

[keishi, 2014/09/11 12:53:26]

Done.

+
     // Used to group a series of resize events. For example, if the user
     // drags a resizer then willStartLiveResize will be called, followed by a
     // sequence of resize events, ending with willEndLiveResize when the user
     // lets go of the resizer.
     virtual void willStartLiveResize() { }
 
     // Called to resize the WebWidget.
     virtual void resize(const WebSize&) { }
 
     // Resizes the unscaled pinch viewport. Normally the unscaled pinch
@@ skipping 175 matching lines @@
     // Informs the WebWidget that the resizer rect changed. Happens for example
     // on mac, when a widget appears below the WebWidget without changing the
     // WebWidget's size (WebWidget::resize() automatically checks the resizer
     // rect.)
     virtual void didChangeWindowResizerRect() { }
 
     // The page background color. Can be used for filling in areas without
     // content.
     virtual WebColor backgroundColor() const { return 0xFFFFFFFF; /* SK_ColorWHITE */ }
 
+    // The currently open popup.
+    virtual WebPagePopup* popup() const { return 0; }

[tkent, 2014/09/05 01:59:04]

nit: |popup| sounds too generic. Maybe |pagePopup|, and please add comment that
this is for calendar / datalist pickers and not for <select>.

[keishi, 2014/09/11 12:53:26]

Done.

+
 protected:
     ~WebWidget() { }
 };
 
 } // namespace blink
 
 #endif