Summary: | AX: Replace AXObjectCache postNotification method boolean arguments with enum values. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Samuel White <samuel_white> | ||||||
Component: | Accessibility | Assignee: | Samuel White <samuel_white> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, eric, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Samuel White
2013-09-17 10:34:29 PDT
Created attachment 212381 [details]
Patch.
Changing postNotification bool argument to enum per "Names" rule #10 in the WebKit coding style guidelines and reviewer feedback.
Comment on attachment 212381 [details] Patch. View in context: https://bugs.webkit.org/attachment.cgi?id=212381&action=review > Source/WebCore/accessibility/AXObjectCache.cpp:723 > + if (object && (postTarget == TargetObservableParent)) extra parens here are not necessary Created attachment 212477 [details]
Updated patch.
Thanks for the feedback Chris. I had added the inner parens because I figured it saved people from having to do a mental lookup of operator precedence. Removed.
Comment on attachment 212477 [details] Updated patch. Clearing flags on attachment: 212477 Committed r156339: <http://trac.webkit.org/changeset/156339> All reviewed patches have been landed. Closing bug. *** Bug 112857 has been marked as a duplicate of this bug. *** |