Summary: | Cannot select certain text on Wikipedia homepage even when zoomed in | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Enrica Casucci <enrica> | ||||
Component: | WebKit2 | Assignee: | Enrica Casucci <enrica> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, mitz, thorton | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | iOS 9.0 | ||||||
Attachments: |
|
Description
Enrica Casucci
2015-10-02 12:23:51 PDT
Created attachment 262346 [details]
Patch
Attachment 262346 [details] did not pass style-queue:
ERROR: Source/WebKit2/UIProcess/WebPageProxy.h:467: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/WebPageProxy.h:474: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/WebPageProxy.h:475: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/WebPageProxy.h:476: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/WebPageProxy.h:479: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm:369: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm:441: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm:452: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm:474: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm:496: Extra space before ( in function call [whitespace/parens] [4]
Total errors found: 10 in 7 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Looks really good. Why can’t we have a regression test? (In reply to comment #3) > Looks really good. Why can’t we have a regression test? I'll add it as soon as Simon has the long press working. Thanks for the review! Committed revision 190568. |