Settings: Users: Fix margins for restrict signins list

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

 <link rel="import" href="chrome://resources/html/polymer.html">
 
 <link rel="import" href="chrome://resources/cr_elements/cr_scrollable_behavior.html">
 <link rel="import" href="chrome://resources/cr_elements/icons.html">
 <link rel="import" href="chrome://resources/html/i18n_behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout-classes.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
 <link rel="import" href="../route.html">
 <link rel="import" href="../settings_shared_css.html">
 
@@ skipping 16 matching lines @@
         background-size: cover;
         border-radius: 20px;
         flex-shrink: 0;
         height: 40px;
         width: 40px;
       }
 
       .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_]]">
         <div class="user layout horizontal center two-line">
           <img class="user-icon" src="[[getProfilePictureUrl_(item)]]">
           <div class="flex user-info">
             <div>[[getUserName_(item)]]</div>
             <template is="dom-if" if="[[shouldShowEmail_(item)]]">
               <div class="secondary">[[item.displayEmail]]</div>
             </template>
           </div>
           <paper-icon-button icon="cr:clear" class="clear-icon"
               on-tap="removeUser_"
               hidden="[[shouldHideCloseButton_(disabled, item.isOwner)]]">
           </paper-icon-button>
         </div>
       </template>
     </div>
   </template>
   <script src="user_list.js"></script>
 </dom-module>