DevTools: minor cleanup for the icons

third_party/WebKit/Source/devtools/front_end/ui/Icon.js

Index: third_party/WebKit/Source/devtools/front_end/ui/Icon.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/Icon.js b/third_party/WebKit/Source/devtools/front_end/ui/Icon.js
index 0a85a871a849c2f8e59d6e7d6e4e48aa5bae57e3..81184630123239aac27cb0a3057b9136b112f839 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/Icon.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/Icon.js
@@ -119,7 +119,6 @@ UI.Icon.Descriptors = {
   'smallicon-file': {x: -64, y: -24, width: 12, height: 14, spritesheet: 'largeicons'},
   'smallicon-file-sync': {x: -76, y: -24, width: 12, height: 14, spritesheet: 'largeicons'},
   'smallicon-search': {x: -234, y: -30, width: 12, height: 12, spritesheet: 'largeicons'},
-  'smallicon-clear-input': {x: -143, y: -96, width: 13, height: 13, spritesheet: 'largeicons'},
   'smallicon-checkmark': {x: -128, y: -109, width: 10, height: 10, spritesheet: 'largeicons'},
 
   'largeicon-longclick-triangle': {x: -290, y: -46, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
@@ -130,7 +129,6 @@ UI.Icon.Descriptors = {
   'largeicon-clear': {x: -64, y: 0, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-rotate-screen': {x: -192, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-phone': {x: -320, y: -96, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
-  'largeicon-block': {x: -32, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-layout-editor': {x: 0, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-foreground-color': {x: -128, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-background-color': {x: -96, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
@@ -175,38 +173,10 @@ UI.Icon.Descriptors = {
   'largeicon-hide-left-sidebar': {x: -192, y: -72, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-show-right-sidebar': {x: -192, y: -96, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-hide-right-sidebar': {x: -192, y: -120, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
-  'largeicon-show-top-sidebar': {
-    x: -288,
-    y: -96,
-    width: 28,
-    height: 24,
-    spritesheet: 'largeicons',
-    isMask: true,
-  },
-  'largeicon-hide-top-sidebar': {
-    x: -256,
-    y: -96,
-    width: 28,
-    height: 24,
-    spritesheet: 'largeicons',
-    isMask: true,
-  },
-  'largeicon-show-bottom-sidebar': {
-    x: -224,
-    y: -144,
-    width: 28,
-    height: 24,
-    spritesheet: 'largeicons',
-    isMask: true,
-  },
-  'largeicon-hide-bottom-sidebar': {
-    x: -256,
-    y: -120,
-    width: 28,
-    height: 24,
-    spritesheet: 'largeicons',
-    isMask: true,
-  },
+  'largeicon-show-top-sidebar': {x: -288, y: -96, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
+  'largeicon-hide-top-sidebar': {x: -256, y: -96, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
+  'largeicon-show-bottom-sidebar': {x: -224, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
+  'largeicon-hide-bottom-sidebar': {x: -256, y: -120, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-navigator-file': {x: -226, y: -72, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-navigator-file-sync': {x: -162, y: -24, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'badge-navigator-file-sync': {x: -322, y: -72, width: 28, height: 24, spritesheet: 'largeicons'},
@@ -216,7 +186,7 @@ UI.Icon.Descriptors = {
   'largeicon-navigator-worker': {x: -322, y: -144, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-navigator-snippet': {x: -226, y: -96, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
   'largeicon-edit': {x: -160, y: -0, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},
-  'largeicon-chevron': {x: -68, y: -143, width: 24, height: 26, spritesheet: 'largeicons', isMask: true},
+  'largeicon-chevron': {x: -68, y: -143, width: 28, height: 24, spritesheet: 'largeicons', isMask: true},

[dgozman, 2017/03/24 16:53:59]

Don't you have to adjust the usages? Or is it magically placed at the right
spot?

[lushnikov, 2017/03/24 16:55:11]

It is Magically placed :)

 
   'mediumicon-manifest': {x: 0, y: -0, width: 16, height: 16, spritesheet: 'resourceicons', isMask: true},
   'mediumicon-service-worker': {x: -20, y: -0, width: 16, height: 16, spritesheet: 'resourceicons', isMask: true},