Improve consistency of nextNodeWithGreaterTabIndex() and previousNodeWithLowerTabIndex().

* nextNodeWithGreaterTabIndex:
  Set |currentTabIndex| to |winningTabIndex|
  They are same in the |if| block.  However |currentTabIndex| looks better.

* previousNodeWithLowerTabIndex:
  Remove isNonKeyboardFocusableShadowHost() check.  It's redandunt because
  shouldVisit() contains it.

BUG=
TEST=none; no behavior changes

Review URL: https://codereview.chromium.org/700313002

git-svn-id: svn://svn.chromium.org/blink/trunk@184889 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/core/page/FocusController.cpp

@@ -549,7 +549,7 @@ static Node* nextNodeWithGreaterTabIndex(Node* start, int tabIndex)
         int currentTabIndex = adjustedTabIndex(&node);
         if (shouldVisit(&node) && currentTabIndex > tabIndex && currentTabIndex < winningTabIndex) {
             winner = &node;
-            winningTabIndex = node.tabIndex();
+            winningTabIndex = currentTabIndex;
         }
     }
 
@@ -563,7 +563,7 @@ static Node* previousNodeWithLowerTabIndex(Node* start, int tabIndex)
     Node* winner = nullptr;
     for (Node* node = start; node; node = NodeTraversal::previous(*node)) {
         int currentTabIndex = adjustedTabIndex(node);
-        if ((shouldVisit(node) || isNonKeyboardFocusableShadowHost(node)) && currentTabIndex < tabIndex && currentTabIndex > winningTabIndex) {
+        if (shouldVisit(node) && currentTabIndex < tabIndex && currentTabIndex > winningTabIndex) {
             winner = node;
             winningTabIndex = currentTabIndex;
         }