Add pixel readback to page popup

public/platform/WebCompositeAndReadbackAsyncCallback.h

Index: public/platform/WebCompositeAndReadbackAsyncCallback.h
diff --git a/public/platform/WebCompositeAndReadbackAsyncCallback.h b/public/platform/WebCompositeAndReadbackAsyncCallback.h
index 45a61be2332878a751a1f8d2b5cbe278f2be46da..4cdcc5bbc2d6a8254f1b4d47175267f57ed13f23 100644
--- a/public/platform/WebCompositeAndReadbackAsyncCallback.h
+++ b/public/platform/WebCompositeAndReadbackAsyncCallback.h
@@ -26,6 +26,8 @@
 #ifndef WebCompositeAndReadbackAsyncCallback_h
 #define WebCompositeAndReadbackAsyncCallback_h
 
+#include "public/platform/WebPoint.h"
+
 class SkBitmap;
 
 namespace blink {
@@ -33,6 +35,14 @@ namespace blink {
 class WebCompositeAndReadbackAsyncCallback {
 public:
     virtual void didCompositeAndReadback(const SkBitmap&) = 0;
+    void setIsPopup(bool isPopup) { m_isPopup = isPopup; };

[tkent, 2014/09/01 05:37:03]

Because this is a public API, please write comments on each of functions.  e.g.
Who should call the function, what happens by calling it, etc.

[jamesr, 2014/09/02 19:09:29]

nothing in platform/ should know or care about what a popup is. why would this
code need to be different from the normal readback paths?

[keishi, 2014/09/04 13:16:20]

Removed. I changed it so I don't stick these data into the callback here.

+    bool isPopup() { return m_isPopup; }
+    void setReadbackPosition(const WebPoint& position) { m_position = position; };
+    WebPoint& readbackPosition() { return m_position; }
+
+private:
+    bool m_isPopup;
+    WebPoint m_position;
 };
 
 } // namespace blink