[ash-md] Adjusts layout of lists with sticky header rows to match specs

ash/common/system/chromeos/network/network_list_md.h

Index: ash/common/system/chromeos/network/network_list_md.h
diff --git a/ash/common/system/chromeos/network/network_list_md.h b/ash/common/system/chromeos/network/network_list_md.h
index bbf8f5567a689e1fd1d33286985eb6a19553dc1a..3df22287d388ff4a70058ae7c51ccc4ba484e57a 100644
--- a/ash/common/system/chromeos/network/network_list_md.h
+++ b/ash/common/system/chromeos/network/network_list_md.h
@@ -21,6 +21,7 @@
 
 namespace views {
 class Label;
+class Separator;
 class View;
 }
 
@@ -91,11 +92,12 @@ class NetworkListViewMd : public NetworkListViewBase,
 
   // Creates a cellular/Wi-Fi header row |view| and adds it to |container()| if
   // necessary and reorders the |container()| placing the |view| at
-  // |child_index|.
-  void UpdateSectionHeaderRow(chromeos::NetworkTypePattern pattern,
-                              bool enabled,
-                              int child_index,
-                              SectionHeaderRowView** view);
+  // |child_index|. Returns the last occupied child index.

[tdanderson, 2016/11/24 23:37:53]

I think the comment you've added should instead read "Returns the index where
the next child should be inserted, i.e., the index directly after the last
inserted child." - that seems to be what you're doing in the implementation.

[varkha, 2016/11/28 17:14:39]

Done.

+  int UpdateSectionHeaderRow(chromeos::NetworkTypePattern pattern,
+                             bool enabled,
+                             int child_index,
+                             SectionHeaderRowView** view,
+                             views::Separator** separator_view);
 
   // network_icon::AnimationObserver:
   void NetworkIconChanged() override;
@@ -107,6 +109,8 @@ class NetworkListViewMd : public NetworkListViewBase,
   views::Label* no_cellular_networks_view_;
   SectionHeaderRowView* cellular_header_view_;
   SectionHeaderRowView* wifi_header_view_;
+  views::Separator* cellular_separator_view_;
+  views::Separator* wifi_separator_view_;
 
   // An owned list of network info.
   std::vector<std::unique_ptr<NetworkInfo>> network_list_;