[Webkit-unassigned] [Bug 73235] [Chromium] Support adding/removing page overlay to WebView
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Dec 1 17:37:35 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=73235
James Robinson <jamesr at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #117519|review? |review-
Flag| |
--- Comment #32 from James Robinson <jamesr at chromium.org> 2011-12-01 17:37:35 PST ---
(From update of attachment 117519)
View in context: https://bugs.webkit.org/attachment.cgi?id=117519&action=review
I thought I left some feedback on this earlier but I don't see it on the bug now. My apologies if you end up getting some comments twice. This is very close!
> Source/WebKit/chromium/WebKit.gyp:263
> + 'public/WebPageOverlayClient.h',
should be WebPageOverlay.h
> Source/WebKit/chromium/public/WebPageOverlay.h:43
> + // Returns dirty rect that needs repaint.
> + virtual WebRect dirtyRect() const = 0;
I think we should remove dirtyRect() completely from this patch and always consider the full viewport to be covered by every overlay.
> Source/WebKit/chromium/src/PageOverlay.cpp:150
> + FloatSize size(m_viewImpl->size().width, m_viewImpl->size().height);
m_viewImpl->size() is a WebSize, which (within WebKit code) can be implicitly converted to an IntSize, which in turn can be implicitly converted to a FloatSize
so this should just be "FloatSize size = m_viewImpl->size();" or "FloatSize size(m_viewImpl->size());" if you prefer that initialization style
> Source/WebKit/chromium/src/PageOverlay.cpp:167
> + if (updateOption == ForceUpdate)
> + m_layer->setNeedsDisplay();
> + else {
> + const WebRect& dirtyRect = m_overlay->dirtyRect();
> + if (!dirtyRect.isEmpty())
> + m_layer->setNeedsDisplayInRect(WebCore::FloatRect(dirtyRect.x, dirtyRect.y, dirtyRect.width, dirtyRect.height));
> + }
i think this can go back to what it previously did
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list