[Chrome OS MD] Update Cast and VPN to follow MD layout

ash/common/system/chromeos/network/vpn_list_view.cc

Index: ash/common/system/chromeos/network/vpn_list_view.cc
diff --git a/ash/common/system/chromeos/network/vpn_list_view.cc b/ash/common/system/chromeos/network/vpn_list_view.cc
index 938a9bd5be588501ea4d741816f2522ee5b366c3..628231dd6a729d0c13d4960cfe86b9f136ba3642 100644
--- a/ash/common/system/chromeos/network/vpn_list_view.cc
+++ b/ash/common/system/chromeos/network/vpn_list_view.cc
@@ -284,8 +284,8 @@ void VPNListNetworkEntry::UpdateFromNetworkState(
     }
 
     SetBorder(
-        views::CreateEmptyBorder(0, kTrayPopupPaddingHorizontal, 0,
-                                 UseMd() ? kTrayPopupButtonEndMargin : 3));
+        views::CreateEmptyBorder(0, UseMd() ? 0 : kTrayPopupPaddingHorizontal,

[bruthig, 2016/11/23 20:15:47]

Do the horizontal insets need to be updated for rows which are not
connected/connecting? i.e. on line 290?

I find it very suspicious that we are setting a border on |this| directly as
well as using a TriView for layout. Ideally it would be one or the other.

[yiyix, 2016/11/24 21:38:16]

I updated the code path such that all border changes are specific to non-md path
only. I also investigated/tried to make non-md path to use tri_view as well,
this change results removing hundred of lines. I think it would make it harder
to merge to the branch and user probably doesn't see this visual change as MD is
turned on. 

[bruthig, 2016/11/24 22:36:29]

Acknowledged.

+                                 0, UseMd() ? kTrayPopupButtonEndMargin : 3));
   } else {
     SetBorder(views::CreateEmptyBorder(0, kTrayPopupPaddingHorizontal, 0, 0));
   }