CrOS Shelf: Address a TODO in the Chrome OS switches

Change-Id: Ic9e61ec64caf8c338ec8e485c23e43c111d344be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108160Reviewed-by: Andrew Xu <andrewxu@chromium.org>
Commit-Queue: Manu Cornet <manucornet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751664}

chromeos/constants/chromeos_switches.cc

@@ -561,12 +561,6 @@ bool ShouldShowShelfHoverPreviews() {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(kShelfHoverPreviews);
 }
 
-bool ShouldShowScrollableShelf() {
-  // TODO(manucornet): Remove this method and simplify conditions depending
-  // on it since it's now always true.
-  return true;
-}
-
 bool ShouldTetherHostScansIgnoreWiredConnections() {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       kTetherHostScansIgnoreWiredConnections);

chromeos/constants/chromeos_switches.h

@@ -230,9 +230,6 @@ COMPONENT_EXPORT(CHROMEOS_CONSTANTS) bool ShouldShowShelfHotseat();
 // on the shelf.
 COMPONENT_EXPORT(CHROMEOS_CONSTANTS) bool ShouldShowShelfHoverPreviews();
 
-// Returns true if we should show a scrollable list of apps in the main shelf.
-COMPONENT_EXPORT(CHROMEOS_CONSTANTS) bool ShouldShowScrollableShelf();
-
 // Returns true if the Chromebook should ignore its wired connections when
 // deciding whether to run scans for tethering hosts. Should be used only for
 // testing.