-
rniwa@webkit.org authored
Reviewed by Darin Adler. EventHandler calls shouldChangeSelection needlessly https://bugs.webkit.org/show_bug.cgi?id=56324 Extracted setSelectionIfNeeded and setNonDirectionalSelectionIfNeeded and avoided calling shouldChangeSelection and setSelection when the existing selection is identical to that of new selection. * page/EventHandler.cpp: (WebCore::setSelectionIfNeeded): Extracted. (WebCore::setNonDirectionalSelectionIfNeeded): Extracted. (WebCore::EventHandler::selectClosestWordFromMouseEvent): Calls a helper function above. (WebCore::EventHandler::selectClosestWordOrLinkFromMouseEvent): Ditto. (WebCore::EventHandler::handleMousePressEventTripleClick): Ditto. (WebCore::EventHandler::handleMousePressEventSingleClick): Ditto. (WebCore::EventHandler::updateSelectionForMouseDrag): Ditto. (WebCore::EventHandler::handleMouseReleaseEvent): Ditto. 2011-03-14 Anton Muhin <antonm@chromium.org> Reviewed by Adam Barth. [v8] Rework object group building. https://bugs.webkit.org/show_bug.cgi?id=55399 Instead of going top-down (from owner to owned elements), go up---from objects to their group ids. That fits better to v8's object grouping model and guarantees that each wrapper belongs to the single group. Alas, this cannot be implemented for one kind of objects---CSSProperties. Part of core GC algorithm and tested extensively by exisiting layout tests. * platform/chromium/test_expectations.txt: Temporary suppress text differences. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@81095 268f45cc-cd09-0410-ab3c-d52691b4dbfc
aea439d1