Summary: | Use PassRefPtr less in CompositeEditCommand | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||||
Component: | WebCore Misc. | Assignee: | Chris Dumez <cdumez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | achristensen, buildbot, commit-queue, darin, ggaren, sam | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Chris Dumez
2017-05-02 19:44:37 PDT
Created attachment 308881 [details]
Patch
Attachment 308881 [details] did not pass style-queue:
ERROR: Source/WebCore/editing/ApplyStyleCommand.cpp:671: 'styleToApply' is incorrectly named. It should be named 'protector' or 'protectedStyle'. [readability/naming/protected] [4]
ERROR: Source/WebCore/editing/ApplyStyleCommand.cpp:951: Multi line control clauses should use braces. [whitespace/braces] [4]
Total errors found: 2 in 22 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 308885 [details]
Patch
Attachment 308885 [details] did not pass style-queue:
ERROR: Source/WebCore/editing/ApplyStyleCommand.cpp:671: 'styleToApply' is incorrectly named. It should be named 'protector' or 'protectedStyle'. [readability/naming/protected] [4]
ERROR: Source/WebCore/editing/ApplyStyleCommand.cpp:951: Multi line control clauses should use braces. [whitespace/braces] [4]
Total errors found: 2 in 22 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 308907 [details]
Patch
Attachment 308907 [details] did not pass style-queue:
ERROR: Source/WebCore/editing/ApplyStyleCommand.cpp:671: 'styleToApply' is incorrectly named. It should be named 'protector' or 'protectedStyle'. [readability/naming/protected] [4]
ERROR: Source/WebCore/editing/ApplyStyleCommand.cpp:951: Multi line control clauses should use braces. [whitespace/braces] [4]
Total errors found: 2 in 22 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 308907 [details] Patch Clearing flags on attachment: 308907 Committed r216128: <http://trac.webkit.org/changeset/216128> All reviewed patches have been landed. Closing bug. |