Fix mash shelf id mapping and removal.

Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org,sky@chromium.org

Committed: https://crrev.com/4875d4b179e246bdc4a903bc3d20c831727eecca
Cr-Commit-Position: refs/heads/master@{#390289}

[msw, 2016-04-27 21:42:08]

Description was changed from

==========
Fix mash shelf id mapping and removal.

BUG=607271
==========

to

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org
==========

[msw, 2016-04-27 21:42:09]

msw@chromium.org changed reviewers:
+ sadrul@chromium.org

[msw, 2016-04-27 21:42:21]

Hey Sadrul, please take a look; thanks!

[sadrul, 2016-04-27 22:49:22]

lgtm. Thanks!

[msw, 2016-04-27 23:18:35]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2016-04-27 23:18:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1932503002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1932503002/1

[msw, 2016-04-27 23:19:50]

Description was changed from

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org
==========

to

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org,sky@chromium.org
==========

[msw, 2016-04-27 23:19:51]

msw@chromium.org changed reviewers:
+ sky@chromium.org

[msw, 2016-04-27 23:20:04]

+Scott for owners, please take a look; thanks!

[commit-bot: I haz the power, 2016-04-27 23:29:25]

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

[commit-bot: I haz the power, 2016-04-27 23:29:26]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[sky, 2016-04-27 23:57:15]

LGTM

[msw, 2016-04-28 03:25:23]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2016-04-28 03:25:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1932503002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1932503002/1

[commit-bot: I haz the power, 2016-04-28 03:32:52]

Description was changed from

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org,sky@chromium.org
==========

to

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org,sky@chromium.org
==========

[commit-bot: I haz the power, 2016-04-28 03:32:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-30 17:15:55]

Patchset 1 (id:??) landed as
https://crrev.com/4875d4b179e246bdc4a903bc3d20c831727eecca
Cr-Commit-Position: refs/heads/master@{#390289}

[commit-bot: I haz the power, 2016-04-30 17:54:00]

Description was changed from

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org,sky@chromium.org
==========

to

==========
Fix mash shelf id mapping and removal.

Add ShelfID to app_id mapping, to complement the inverse map.
(these are 1-to-1 mappings, but two maps eases usage)

Remove mappings when shelf items are removed.
Implement the ShelfDelegate interface mapping functions.

TODO: Add unit/shell testing of ShelfDelegateMus.

BUG=607271
TEST=|ShelfDelegaeMus::app_id_to_shelf_id_| shrinks as appropriate.
R=sadrul@chromium.org,sky@chromium.org

Committed: https://crrev.com/4875d4b179e246bdc4a903bc3d20c831727eecca
Cr-Commit-Position: refs/heads/master@{#390289}
==========