Adding a ScrollView for IntentPickerBubbleView

chrome/browser/chromeos/arc/arc_navigation_throttle.h

Index: chrome/browser/chromeos/arc/arc_navigation_throttle.h
diff --git a/chrome/browser/chromeos/arc/arc_navigation_throttle.h b/chrome/browser/chromeos/arc/arc_navigation_throttle.h
index 781817039ee42e2f5926586b747e7a8ed34d0c3e..0690af2a6a8951b2f5219f952ffef5cc7425dc5b 100644
--- a/chrome/browser/chromeos/arc/arc_navigation_throttle.h
+++ b/chrome/browser/chromeos/arc/arc_navigation_throttle.h
@@ -23,10 +23,6 @@ class NavigationHandle;
 
 namespace arc {
 
-// Restricts the amount of apps displayed to the user without the need of a
-// ScrollView.
-constexpr size_t kMaxAppResults = 3;
-
 // A class that allow us to retrieve ARC app's information and handle URL
 // traffic initiated on Chrome browser, either on Chrome or an ARC's app.
 class ArcNavigationThrottle : public content::NavigationThrottle {
@@ -43,6 +39,10 @@ class ArcNavigationThrottle : public content::NavigationThrottle {
     SIZE,
   };
 
+  // Restricts the amount of apps displayed to the user without the need of a
+  // ScrollView.
+  enum { kMaxAppResults = 3 };

[sky, 2016/07/21 22:57:06]

enum style is ALL_CAPS (like CloseReason above). That said, why are using an
enum here at all?

[Luis Héctor Chávez, 2016/07/21 22:59:59]

https://groups.google.com/a/chromium.org/forum/#!msg/chromium-dev/hDqJ4KBVqog...

+
   using NameAndIcon = std::pair<std::string, gfx::Image>;
   using ShowDisambigDialogCallback =
       base::Callback<void(content::NavigationHandle* handle,