Adding/renaming a folder in the bookmark manager should not hide the icon.

chrome/browser/resources/bookmark_manager/css/bmm.css

 /* Copyright (c) 2012 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. */
 
 <include src="../search_header.css">
 
 html,
 body {
   cursor: default;
   height: 100%;
@@ skipping 38 matching lines @@
 
 list .label {
   -webkit-padding-start: 0;
   align-items: center;
   display: flex;
 }
 
 list .label-img-wrapper {
   display: inline-block; /* Makes the image start-align in RTL. */
   width: 20px;
+  z-index: -1;

[Peter Kasting, 2016/10/26 01:49:57]

Without this, the input field, which has a -webkit-margin-start of -4px (see
line 142), looks truncated on the left where it overlaps this div, even though
the div "appears" to be empty in this region.

I don't know exactly what draws the gradient background on selected BMM rows,
but that background is what we see in this region instead.  Maybe it's getting
separately drawn on each individual div.

We don't want to fix this by removing the -4px margin, or the text will shift
horizontally when we start/stop editing.

[Dan Beam, 2016/10/27 03:56:16]

Acknowledged.

 }
 
 <if expr="not is_macosx and not is_ios">
 html[dir=rtl] list > .folder .label-img-wrapper > * {
   transform: scaleX(-1);
 }
 </if>
 
 list .label-text {
   color: black;
@@ skipping 59 matching lines @@
   display: block;
 }
 
 html[dir=rtl] list > * > button {
   left: 3px;
   right: auto;
 }
 
 /* Edit mode */
 
-list [editing] .label input,
+list [editing] .label-text input,

[Peter Kasting, 2016/10/26 01:49:57]

This change should have no effect, nor should the one on line 166, but they were
made to be consistent with line 176.

[Dan Beam, 2016/10/27 03:56:16]

Acknowledged.

 list [editing] .url input {
   -webkit-margin-end: 4px;
   -webkit-margin-start: -4px;
   -webkit-padding-end: 3px;
   -webkit-padding-start: 3px;
   box-sizing: content-box;
   font-family: inherit;
   font-size: inherit;
   font-weight: inherit;
   /* Do not inherit the line-height. */
   line-height: normal;
   margin-bottom: 0;
   margin-top: 0;
   min-height: 0;
   text-decoration: none;
   vertical-align: baseline;
 }
 
 .tree-item [editing] input {
   line-height: normal;
   margin: 0;
   min-height: 0;
   padding: 1px 0;
 }
 
 <if expr="is_macosx">
-list .label input,
+list .label-text input,
 list .url input {
   outline: none;
 }
 </if>
 
 list > [editing] {
   overflow: visible;
 }
 
-list [editing] .label,
+list [editing] .label-text,

[Peter Kasting, 2016/10/26 01:49:57]

I made this change speculatively.  Due to
https://codereview.chromium.org/2350473003 , whatever purpose the "overflow:
visible" style here served was no effective.  Before that change, .label was the
direct parent of the <input> element; afterwards, .label-text was introduced to
fill that role, and given its own "overflow: hidden" style (see line 71), which
would override this.

So this aims to return to the previous behavior.  But I don't know what effect
toggling these overflow values actually has in this specific case, so I'm not
sure how to test that this is really fixing a bug.  Help :(

[Dan Beam, 2016/10/27 03:56:17]

¯\_(ツ)_/¯

[Peter Kasting, 2016/10/27 06:00:26]

:(

I'm inclined to rip out the overflow fiddling entirely and see what breaks,
except I wouldn't notice, and months later it'd end up being tasked to me to
fix. :P

This whole file needs way more "Why do we have this" comments.

[Dan Beam, 2016/10/27 17:27:55]

i mean, i could find out, but this code is basicallly deprecated in my mind and
i'd just prefer to focus on its replacement rather than understanding old code
with all authors gone

 list [editing] .url,
 list [editing] > * {
   overflow: visible;
 }
 
 list [editing] .url {
   -webkit-padding-start: 5px;
 }
 
 list [editing] input {
@@ skipping 196 matching lines @@
 
   list [editing] input,
   .tree-item [editing] input {
     padding: 3px 0;
   }
 
   .tree-row .expand-icon {
     top: 6px;
   }
 }