[Webkit-unassigned] [Bug 71579] Exposing getRect function in WebNode for chromium

[bugzilla-daemon at webkit.org]

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





--- Comment #14 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2011-11-08 14:21:46 PST ---
(In reply to comment #11)
> (In reply to comment #10)
> > (In reply to comment #7)
> > > Is it fine for the function to call layout() internally?  If so, I'd prefer that we do that -- but I defer to your judgement, as you obviously have much more experience with all things WebKit API.
> > 
> > I believe that most of our existing APIs that return layout metrics do not internally force layout to be up-to-date.
> 
> Fair enough.  I saw this change [1] get committed recently that made me think we might want to force layout in such methods.
> 
> [1] http://trac.webkit.org/changeset/94586/trunk/Source/WebKit/chromium/src/WebNode.cpp

Hmm... hmm... well, Document::updateLayout() is cheap to call when there is no work to be done, so perhaps you should consider calling that.  I really haven't thought carefully enough about this issue.  Hmm...

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