[MD Bookmarks] Fix sidebar issues.

chrome/browser/resources/md_bookmarks/app.js

 // Copyright 2016 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.
 
 Polymer({
   is: 'bookmarks-app',
 
   behaviors: [
     bookmarks.StoreClient,
   ],
@@ skipping 64 matching lines @@
   /**
    * Set up the splitter and set the initial width from localStorage.
    * @private
    */
   initializeSplitter_: function() {
     var splitter = this.$.splitter;
     cr.ui.Splitter.decorate(splitter);
     var splitterTarget = this.$.sidebar;
 
     // The splitter persists the size of the left component in the local store.
-    if ('treeWidth' in window.localStorage) {
-      splitterTarget.style.width = window.localStorage['treeWidth'];
-      this.sidebarWidth_ = splitterTarget.getComputedStyleValue('width');
+    if (LOCAL_STORAGE_TREE_WIDTH_KEY in window.localStorage) {
+      splitterTarget.style.width =
+          window.localStorage[LOCAL_STORAGE_TREE_WIDTH_KEY];
     }
+    this.sidebarWidth_ = splitterTarget.getComputedStyleValue('width');
 
     splitter.addEventListener('resize', function(e) {
-      window.localStorage['treeWidth'] = splitterTarget.style.width;
-      // TODO(calamity): This only fires when the resize is complete. This
-      // should be updated on every width change.
+      window.localStorage[LOCAL_STORAGE_TREE_WIDTH_KEY] =
+          splitterTarget.style.width;
       this.updateSidebarWidth_();
     }.bind(this));
 
+    splitter.addEventListener('dragmove', function(e) {
+      this.updateSidebarWidth_();
+    }.bind(this));

[Dan Beam, 2017/04/11 01:27:02]

nit: could also be

splitter.addEventListener('dragmove', this.updateSidebarWidth_.bind(this));

if you don't care about omitting the param

[calamity, 2017/04/11 03:31:39]

Done.

+
     window.addEventListener('resize', this.boundUpdateSidebarWidth_);
   },
 
   /** @private */
   updateSidebarWidth_: function() {
     this.sidebarWidth_ = this.$.sidebar.getComputedStyleValue('width');
   },
 
   /** @private */
   searchTermChanged_: function() {
     if (!this.searchTerm_)
       return;
 
     chrome.bookmarks.search(this.searchTerm_, function(results) {
       var ids = results.map(function(node) {
         return node.id;
       });
       this.dispatch(bookmarks.actions.setSearchResults(ids));
     }.bind(this));
   },
 
   /** @private */
   closedFoldersStateChanged_: function() {
     window.localStorage[LOCAL_STORAGE_CLOSED_FOLDERS_KEY] =
         JSON.stringify(Array.from(this.closedFoldersState_));
   },
 });