Per Darin's review comment in the bug 154630, use more references in FocusNavigationScope.
Created attachment 272147 [details] Cleanup
Attachment 272147 [details] did not pass style-queue: ERROR: Source/WebCore/ChangeLog:20: Line contains tab character. [whitespace/tab] [5] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 272148 [details] Did one more rename
Comment on attachment 272148 [details] Did one more rename View in context: https://bugs.webkit.org/attachment.cgi?id=272148&action=review r=me > Source/WebCore/page/FocusController.cpp:170 > +FocusNavigationScope FocusNavigationScope::scopeOwnedByShadowHost(Node& node) This should even take an Element& in. Casting inside without check is terrible.
Comment on attachment 272148 [details] Did one more rename View in context: https://bugs.webkit.org/attachment.cgi?id=272148&action=review >> Source/WebCore/page/FocusController.cpp:170 >> +FocusNavigationScope FocusNavigationScope::scopeOwnedByShadowHost(Node& node) > > This should even take an Element& in. Casting inside without check is terrible. Will fix. Thanks for the review!
Committed r197055: <http://trac.webkit.org/changeset/197055>