[Webkit-unassigned] [Bug 56770] Speed up HitTestResult

[bugzilla-daemon at webkit.org]

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


Dimitri Glazkov (Google) <dglazkov at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #86365|review?                     |review-
               Flag|                            |




--- Comment #3 from Dimitri Glazkov (Google) <dglazkov at chromium.org>  2011-03-21 14:36:16 PST ---
(From update of attachment 86365)
View in context: https://bugs.webkit.org/attachment.cgi?id=86365&action=review

Looks reasonable, aside from a style nitpick.

> Source/WebCore/rendering/HitTestResult.cpp:583
> +        for (NodeSet::const_iterator it = other.m_rectBasedTestResult->begin(),
> +                     last = other.m_rectBasedTestResult->end();

The indent is weird here. Come to the dark side... Put it on one line :)

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