DevTools -> Memory Tab -> Allowing to edit a title of a heap snapshot

third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 623 matching lines @@
   _updateStatus(event) {
     var statusUpdate = event.data;
     if (statusUpdate.subtitle !== null) {
       this._subtitleElement.textContent = statusUpdate.subtitle || '';
       this._titlesElement.classList.toggle('no-subtitle', !statusUpdate.subtitle);
     }
     if (typeof statusUpdate.wait === 'boolean' && this.listItemElement)
       this.listItemElement.classList.toggle('wait', statusUpdate.wait);
   }
 
+  /**
+ * @override

[alph, 2017/07/06 02:09:07]

the padding is off

+ * @return {boolean}
+ */
+  ondblclick(event) {

[alph, 2017/07/06 02:09:07]

please annotate the argument

+    if (this._editing)
+      return false;
+    if (this._startEditing(/** @type {!Element} */ (event.target)))

[alph, 2017/07/06 02:09:07]

no need for an 'if' as it always returns false.

+      return false;
+    return false;
+  }
+
+

[alph, 2017/07/06 02:09:07]

remove the extra line

+  _startEditing(eventTarget) {

[alph, 2017/07/06 02:09:07]

please add annotation.

+    var container = eventTarget.enclosingNodeOrSelfWithClass('title-container');
+    if (container) {

[alph, 2017/07/06 02:09:07]

we prefer early return to keep the code indents minimal, i.e.:

if (!container)
  return;

+      container.textContent = this.profile.title;
+      var config = new UI.InplaceEditor.Config(this._editingCommitted.bind(this), this._editingCancelled.bind(this));
+      this._editing = UI.InplaceEditor.startEditing(container, config);
+      // this.listItemElement.getComponentSelection().selectAllChildren(textNodeElement);

[alph, 2017/07/06 02:09:07]

please do not add commented out code.

+    }
+  }
+
+  _editingCommitted(container, element, newTitle) {

[alph, 2017/07/06 02:09:07]

annotate please

+    delete this._editing;
+
+    this.profile.setTitle(newTitle);

[alph, 2017/07/06 02:09:07]

the third argument is in fact an oldTitle.

+    this.updateTitle();
+  }
+
+  _editingCancelled(element, context) {

[alph, 2017/07/06 02:09:07]

annotate please

+    delete this._editing;
+
+    // Need to restore attributes structure.

[alph, 2017/07/06 02:09:07]

no need for a comment.

+    this.updateTitle();
+  }
+
+  updateTitle() {

[alph, 2017/07/06 02:09:07]

make it private, i.e. _updateTitle

+    if (this._titleContainer.getElementsByClassName('save-link').length === 0)
+      this._createSaveLink();

[alph, 2017/07/06 02:09:07]

I wonder, why do you need to create a save link? It seems to be unrelated.

+    this._titleElement.textContent = this.profile.title;
+  }
+
   dispose() {
     this.profile.removeEventListener(Profiler.ProfileHeader.Events.UpdateStatus, this._updateStatus, this);
     this.profile.removeEventListener(Profiler.ProfileHeader.Events.ProfileReceived, this._onProfileReceived, this);
   }
 
   /**
    * @override
    * @return {boolean}
    */
   onselect() {
@@ skipping 161 matching lines @@
    * @param {string} actionId
    * @return {boolean}
    */
   handleAction(context, actionId) {
     var panel = UI.context.flavor(Profiler.JSProfilerPanel);
     console.assert(panel && panel instanceof Profiler.JSProfilerPanel);
     panel.toggleRecord();
     return true;
   }
 };