[DevTools] Beautified device counter icon.

Source/devtools/front_end/components/InspectorView.js

 /*
  * Copyright (C) 2011 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 45 matching lines @@
     var headerElement = this._tabbedPane.headerElement();
     headerElement.parentElement.insertBefore(this._toolbarElement, headerElement);
 
     this._leftToolbarElement = this._toolbarElement.createChild("div", "toolbar-controls-left");
     this._toolbarElement.appendChild(headerElement);
     this._rightToolbarElement = this._toolbarElement.createChild("div", "toolbar-controls-right");
 
     if (WebInspector.experimentsSettings.devicesPanel.isEnabled()) {
         this._remoteDeviceCountElement = this._rightToolbarElement.createChild("div", "hidden");
         this._remoteDeviceCountElement.addEventListener("click", this.showViewInDrawer.bind(this, "devices"), false);
+        this._remoteDeviceCountElement.id = "remote-device-count";
         WebInspector.inspectorFrontendEventSink.addEventListener(WebInspector.InspectorView.Events.DeviceCountChanged, this._onDeviceCountChanged, this);
     }
 
     this._errorWarningCountElement = this._rightToolbarElement.createChild("div", "hidden");
     this._errorWarningCountElement.id = "error-warning-count";
 
     this._closeButtonToolbarItem = document.createElementWithClass("div", "toolbar-close-button-item");
     var closeButtonElement = this._closeButtonToolbarItem.createChild("div", "close-button");
     closeButtonElement.addEventListener("click", InspectorFrontendHost.closeWindow.bind(InspectorFrontendHost), true);
     this._rightToolbarElement.appendChild(this._closeButtonToolbarItem);
@@ skipping 382 matching lines @@
         var commaString = errors && warnings ? ", " : "";
         this._errorWarningCountElement.title = errorString + commaString + warningString;
         this._tabbedPane.headerResized();
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onDeviceCountChanged: function(event)
     {
         var count = /** @type {number} */(event.data);

[apavlov, 2014/05/29 13:26:11]

A whitespace between '*/' and '(', while we are here

[Dmitry Zvorygin, 2014/05/29 14:55:44]

Done.

+        if (count === this.deviceCount_)
+            return;
+        this.deviceCount_ = count;
         this._remoteDeviceCountElement.classList.toggle("hidden", !count);
         this._remoteDeviceCountElement.removeChildren();
         this._createImagedCounterElementIfNeeded(this._remoteDeviceCountElement, count, "device-count", "device-icon-small");
         this._remoteDeviceCountElement.title = WebInspector.UIString(((count > 1) ? "%d devices found" : "%d device found"), count);
         this._tabbedPane.headerResized();
     },
 
     __proto__: WebInspector.VBox.prototype
 };
 
@@ skipping 67 matching lines @@
             window.addEventListener("scroll", this._onScrollBound, false);
         else
             window.removeEventListener("scroll", this._onScrollBound, false);
 
         WebInspector.VBox.prototype.doResize.call(this);
         this._onScroll();
     },
 
     __proto__: WebInspector.VBox.prototype
 };