Refactor AddScrollListItem() in system menu detailed views

ash/system/tray/tray_popup_utils.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 ASH_SYSTEM_TRAY_TRAY_POPUP_UTILS_H_
 #define ASH_SYSTEM_TRAY_TRAY_POPUP_UTILS_H_
 
 #include <memory>
 
 #include "ash/login_status.h"
@@ skipping 36 matching lines @@
   //
   // The START and END containers have a fixed minimum width but can grow into
   // the CENTER container if space is required and available.
   //
   // The CENTER container has a flexible width.
   static TriView* CreateDefaultRowView();
 
   // Creates a container view to be used by system menu sub-section header rows.
   // The caller takes over ownership of the created view.
   //
-  // The returned view consists of 2 regions: CENTER, and END having the same
-  // properties as when using |CreateMultiTargetRowView|. The START container is
-  // hidden.
-  // The END container has a fixed minimum width but can grow into the CENTER
-  // container if space is required and available. The CENTER container has a
-  // flexible width.
-  static TriView* CreateSubHeaderRowView();
+  // The returned view contains at least CENTER and END regions having the same
+  // properties as when using |CreateMultiTargetRowView|. |start_visible|
+  // determines whether the START region should be visible or not. If START is
+  // not visible, extra padding is added to the left of the contents.
+  //
+  // The START (if visible) and END containers have a fixed minimum width but
+  // can grow into the CENTER container if space is required and available. The
+  // CENTER container has a flexible width.
+  //
+  // TODO(mohsen): Merge this into TrayDetailsView::AddScrollListSubHeader()
+  // once network and VPN alse use TrayDetailsView::AddScrollListSubHeader().
+  static TriView* CreateSubHeaderRowView(bool start_visible);
 
   // Creates a view containing only a label (corresponding to |message_id|),
   // which is to be inserted as a non-targetable row within a system menu
   // detailed view (e.g., the "Scanning for devices..." message that can appear
   // at the top of the Bluetooth detailed view). The caller takes over ownership
   // of the created view.
   static views::View* CreateInfoLabelRowView(int message_id);
 
   // Creates a container view to be used by system menu rows that want to embed
   // a targetable area within one (or more) of the containers OR by any row
@@ skipping 155 matching lines @@
   // |ink_drop_style|.
   static gfx::Rect GetInkDropBounds(TrayPopupInkDropStyle ink_drop_style,
                                     const views::View* host);
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(TrayPopupUtils);
 };
 
 }  // namespace ash
 
 #endif  // ASH_SYSTEM_TRAY_TRAY_POPUP_UTILS_H_