Ensure that Chrome is visible in the dialog by default

chrome/browser/chromeos/arc/intent_helper/arc_navigation_throttle.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_NAVIGATION_THROTTLE_H_
 #define CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_NAVIGATION_THROTTLE_H_
 
 #include <memory>
 #include <string>
 #include <utility>
@@ skipping 54 matching lines @@
 
   static bool ShouldOverrideUrlLoadingForTesting(const GURL& previous_url,
                                                  const GURL& current_url);
 
   // Finds |selected_app_package| from the |handlers| array and returns the
   // index. If the app is not found, returns |handlers.size()|.
   static size_t GetAppIndex(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
       const std::string& selected_app_package);
 
+  // Swaps Chrome app with any app in row |kMaxAppResults-1| iff its index is

[Yusuke Sato, 2016/11/05 01:19:26]

copied the comment from .cc as-is.

+  // bigger, thus ensuring the user can always see Chrome without scrolling.
+  // When swap is needed, fills |out_indices| and returns true. If |handlers|
+  // do not have Chrome, returns false.
+  static bool IsSwapElementsNeeded(
+      const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
+      std::pair<size_t, size_t>* out_indices);
+
   static bool IsAppAvailableForTesting(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers);
   static size_t FindPreferredAppForTesting(
       const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers);
-  static bool IsSwapElementsNeededForTesting(
-      const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
-      std::pair<size_t, size_t>* out_indices);
 
  private:
   // content::Navigation implementation:
   NavigationThrottle::ThrottleCheckResult WillStartRequest() override;
   NavigationThrottle::ThrottleCheckResult WillRedirectRequest() override;
 
   NavigationThrottle::ThrottleCheckResult HandleRequest();
   void OnAppCandidatesReceived(
       mojo::Array<mojom::IntentHandlerInfoPtr> handlers);
   void OnAppIconsReceived(
@@ skipping 19 matching lines @@
 
   // This has to be the last member of the class.
   base::WeakPtrFactory<ArcNavigationThrottle> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcNavigationThrottle);
 };
 
 }  // namespace arc
 
 #endif  // CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_NAVIGATION_THROTTLE_H_