Settings: Users: Fix margins for restrict signins list

chrome/browser/resources/settings/people_page/user_list.html

Index: chrome/browser/resources/settings/people_page/user_list.html
diff --git a/chrome/browser/resources/settings/people_page/user_list.html b/chrome/browser/resources/settings/people_page/user_list.html
index 230c4ffe555795336fa77db6cf81ddb0fa61aadb..96939089b0bc7bff5cb9c2f64c78167c0fc257a1 100644
--- a/chrome/browser/resources/settings/people_page/user_list.html
+++ b/chrome/browser/resources/settings/people_page/user_list.html
@@ -34,6 +34,11 @@
       .user-info {
         -webkit-padding-start: 20px;
       }
+
+      paper-icon-button {
+        -webkit-margin-end: 0;
+        -webkit-margin-start: var(--cr-icon-button-margin-start);

[dpapad, 2017/06/23 23:31:28]

--cr-icon-button-margin-start is defined in shared_vars_css.html. Even if we do
depend on it transitively from settings_shared_css.html, we should import it
directly, no (include-what-you-use principle).

https://cs.chromium.org/chromium/src/ui/webui/resources/cr_elements/shared_va...

[stevenjb, 2017/06/23 23:37:46]

We do not currently have any examples of a direct include of
shared_vars_css.html in resources/settings currently (or even cr_shared_style
where we include settings_shared_css). I'm not sure I have an opinion as to
whether or not we should change that, but it doesn't seem like we should change
it just here.

[dpapad, 2017/06/23 23:49:27]

Ok, I had not realized that was the case. Agreed let's not change the pattern
here. I also think that this eventually be affected by the new deprecation
warning (tracked by
https://bugs.chromium.org/p/chromium/issues/detail?id=735633).
settings_vars_css.html will have to change because of it IIUC.

+      }
     </style>
     <div class="user-list" scrollable>
       <template is="dom-repeat" items="[[users_]]">