DevTools: track more elements panel user actions

third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * 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.
@@ skipping 2070 matching lines @@
 
   /**
    * @override
    */
   onattach() {
     this.updateTitle();
 
     this.listItemElement.addEventListener('mousedown', this._mouseDown.bind(this));
     this.listItemElement.addEventListener('mouseup', this._resetMouseDownElement.bind(this));
     this.listItemElement.addEventListener('click', this._mouseClick.bind(this));
+    this.listItemElement.addEventListener('copy', this._clipboardCopy.bind(this));

[pfeldman, 2017/02/08 19:04:48]

You just created a lot of listeners for no good reason. Place one on the
sidebar's element instead.

[chenwilliam, 2017/02/10 22:53:38]

Done.

   }
 
   /**
    * @param {!Event} event
    */
   _mouseDown(event) {
     if (this._parentPane) {
       this._parentPane._mouseDownTreeElement = this;
       this._parentPane._mouseDownTreeElementIsName =
           this.nameElement && this.nameElement.isSelfOrAncestor(event.target);
       this._parentPane._mouseDownTreeElementIsValue =
           this.valueElement && this.valueElement.isSelfOrAncestor(event.target);
     }
   }
 
   _resetMouseDownElement() {
     if (this._parentPane) {
       delete this._parentPane._mouseDownTreeElement;
       delete this._parentPane._mouseDownTreeElementIsName;
       delete this._parentPane._mouseDownTreeElementIsValue;
     }
   }
 
+  _clipboardCopy() {
+    Host.userMetrics.actionTaken(Host.UserMetrics.Action.StyleRuleCopied);
+  }
+
   /**
    * @override
    */
   onexpand() {
     this._updateExpandElement();
   }
 
   /**
    * @override
    */
@@ skipping 956 matching lines @@
   }
 
   /**
    * @override
    * @return {!UI.ToolbarItem}
    */
   item() {
     return this._button;
   }
 };