[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 #8 from Vsevolod Vlasov <vsevik at chromium.org>  2011-11-29 14:38:16 PST ---
(From update of attachment 116995)
View in context: https://bugs.webkit.org/attachment.cgi?id=116995&action=review

> Source/WebKit/chromium/src/WebFrameImpl.cpp:2067
> +            viewImpl()->pageOverlays()->paintWebFrame(gc);

I think it might worth noting that curently in non-composited case pageOverlay does not work correctly. It is possible that when paintWithContext is called only a part of page was invalidated (dirtyRect?), so pageOverlay can only paint in this part. 
E.g. if you run something like

var d = document.createElement("div"); document.body.appendChild(d);  d.style.height = "100px"; d.style.width = "500px"; d.textContent = "some text"; d.style.position = "absolute";
var left = 0; function upd() {left += 5; d.style.left = left + "px"; setTimeout(upd.bind(this), 1000)} 
upd();

in inspector console and try to highlight this div from inspector, it will correctly highlight moving div, but the tooltip with sizes will not be redrawn.

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