DevTools: define icon location with row and column

DevTools: define icon location with row and column

This patch:
- moves icons from {x, y, width, height} coordinates into
  {row, column} coordinates.
- removes spritesheet image sizes from inspectorCommon.css. This
  turned out to be possible since icon's background/mask-image
  has default size of the image.

This solves multiple issues:
- it's easy to convert between icon coordinates and spritesheets
  (coordinate system is added to SVG files: http://imgur.com/a/rSLSN)
- it discourages using non-standard icon sizes
- resizing spritesheet is as easy as changing its size in inkscape

BUG=700249

[lushnikov, 2017-03-25 09:36:23]

Description was changed from

==========
DevTools: define icon location with row and column

This patch:
- moves icons from {x, y, width, height} coordinates into
  {row, column} coordinates.
- removes spritesheet image sizes from inspectorCommon.css. This
  turned out to be possible since icon's background/mask-image
  has default size of the image.

This solves multiple issues:
- it's easy to convert between icon coordinates and spritesheets
  (coordinate system is added to SVG files)
- it discourages abusing of icon sizes - icon could be only
  part of a cell
- resizing spritesheet is as easy as changing its size in inkscape

BUG=700249
==========

to

==========
DevTools: define icon location with row and column

This patch:
- moves icons from {x, y, width, height} coordinates into
  {row, column} coordinates.
- removes spritesheet image sizes from inspectorCommon.css. This
  turned out to be possible since icon's background/mask-image
  has default size of the image.

This solves multiple issues:
- it's easy to convert between icon coordinates and spritesheets
  (coordinate system is added to SVG files: http://imgur.com/a/rSLSN)
- it discourages abusing of icon sizes - icon could be only
  part of a cell
- resizing spritesheet is as easy as changing its size in inkscape

BUG=700249
==========

[lushnikov, 2017-03-25 09:37:00]

lushnikov@chromium.org changed reviewers:
+ dgozman@chromium.org

[lushnikov, 2017-03-25 09:38:03]

please, take a look

This is the last patch towards icons subsystem improvement.

[lushnikov, 2017-03-25 09:38:09]

The CQ bit was checked by lushnikov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-25 09:38:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[lushnikov, 2017-03-25 09:38:40]

Description was changed from

==========
DevTools: define icon location with row and column

This patch:
- moves icons from {x, y, width, height} coordinates into
  {row, column} coordinates.
- removes spritesheet image sizes from inspectorCommon.css. This
  turned out to be possible since icon's background/mask-image
  has default size of the image.

This solves multiple issues:
- it's easy to convert between icon coordinates and spritesheets
  (coordinate system is added to SVG files: http://imgur.com/a/rSLSN)
- it discourages abusing of icon sizes - icon could be only
  part of a cell
- resizing spritesheet is as easy as changing its size in inkscape

BUG=700249
==========

to

==========
DevTools: define icon location with row and column

This patch:
- moves icons from {x, y, width, height} coordinates into
  {row, column} coordinates.
- removes spritesheet image sizes from inspectorCommon.css. This
  turned out to be possible since icon's background/mask-image
  has default size of the image.

This solves multiple issues:
- it's easy to convert between icon coordinates and spritesheets
  (coordinate system is added to SVG files: http://imgur.com/a/rSLSN)
- it discourages using non-standard icon sizes
- resizing spritesheet is as easy as changing its size in inkscape

BUG=700249
==========

[commit-bot: I haz the power, 2017-03-25 11:23:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-25 11:23:39]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2017-03-25 18:45:15]

Closing this patchset in favor of https://codereview.chromium.org/2776893002/