[Webkit-unassigned] [Bug 127675] [EFL][WK2] Add a logic for checking multi touch in GestureRecognizer::noGesture

[bugzilla-daemon at webkit.org]

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





--- Comment #11 from Sanghyup Lee <sh53.lee at samsung.com>  2014-01-27 17:12:39 PST ---
(From update of attachment 222316)
View in context: https://bugs.webkit.org/attachment.cgi?id=222316&action=review

>> Source/WebKit2/UIProcess/API/efl/GestureRecognizer.cpp:359
>> +        switch (WKArrayGetSize(WKTouchEventGetTouchPoints(eventRef))) {
> 
> You can make WKTouchEventGetTouchPoints() as a variable because it is used multiple times.

Ok, I fixed. thanks

>> Source/WebKit2/UIProcess/API/efl/GestureRecognizer.cpp:361
>> +            break;
> 
> I think it is better to move codes for case 1 here, instead of "break" for readability.

ditto

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