[Webkit-unassigned] [Bug 73235] [Chromium] Support adding/removing page overlay to WebView

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=73235





--- Comment #33 from James Robinson <jamesr at chromium.org>  2011-12-01 17:38:45 PST ---
(From update of attachment 117519)
View in context: https://bugs.webkit.org/attachment.cgi?id=117519&action=review

> Source/WebKit/chromium/public/WebView.h:416
> +    // graphical appearance of the WebView. WebPageOverlayClient paints the

WebPageOverlayClient -> WebPageOverlay

> Source/WebKit/chromium/src/PageOverlay.cpp:182
> +    // WebPageOverlayClient does the actual painting of the overlay.

WebPageOverlayClient -> WebPageOverlay

> Source/WebKit/chromium/src/WebDevToolsAgentImpl.cpp:275
> +// WebPageOverlayClient

WebPageOverlayClient -> WebPageOverlay

> Source/WebKit/chromium/src/WebDevToolsAgentImpl.h:100
> +    // WebPageOverlayClient

WebPageOverlayClient -> WebPageOverlay

> Source/WebKit/chromium/src/WebDevToolsAgentImpl.h:102
> +    virtual WebRect dirtyRect() const;

remove please

> Source/WebKit/chromium/src/WebViewImpl.h:96
> +class WebPageOverlayClient;

WebPageOverlayClient -> WebPageOverlay, or remove this forward declaration if it isn't needed

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.