Bluez: Move released profiles to a separate map.

Bluez: Move released profiles to a separate map.

This addresses an edge case where released profiles may not have been
removed before Shutdown. See issue for details.

BUG=604710
Committed: https://crrev.com/3bfed672ec31b7f51390da4c6ba260942ea8df25
Cr-Commit-Position: refs/heads/master@{#389242}

[stevenjb, 2016-04-22 17:48:56]

stevenjb@chromium.org changed reviewers:
+ rkc@chromium.org, scheib@chromium.org

[stevenjb, 2016-04-22 17:49:07]

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

[commit-bot: I haz the power, 2016-04-22 17:49:38]

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

[rkc, 2016-04-22 19:37:17]

lgtm

nit: This can also be done by re-using the profile vector to store a pair of
profile/bool values. The bool indicating whether this profile has been released
yet.

It only saves a bit of space and may be more complicated to maintain, so either
way works for me. Feel free to go with either.

[stevenjb, 2016-04-22 19:49:17]

On 2016/04/22 19:37:17, Rahul Chaturvedi wrote:
> lgtm
> 
> nit: This can also be done by re-using the profile vector to store a pair of
> profile/bool values. The bool indicating whether this profile has been
released
> yet.
> 
> It only saves a bit of space and may be more complicated to maintain, so
either
> way works for me. Feel free to go with either.

If there were a lot of these, the bool per entry would exceed the extra overhead
of a second map (2 words). Otherwise the overhead is insignificant and worth
avoiding the extra complexity.

[commit-bot: I haz the power, 2016-04-22 20:25:33]

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

[commit-bot: I haz the power, 2016-04-22 20:25:34]

Dry run: This issue passed the CQ dry run.

[stevenjb, 2016-04-22 20:32:03]

The CQ bit was checked by stevenjb@chromium.org

[commit-bot: I haz the power, 2016-04-22 20:32:41]

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

[commit-bot: I haz the power, 2016-04-22 20:48:37]

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

[commit-bot: I haz the power, 2016-04-22 20:48:38]

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...)

[stevenjb, 2016-04-22 21:00:14]

stevenjb@chromium.org changed reviewers:
+ ortuno@chromium.org

[stevenjb, 2016-04-22 21:00:15]

scheib@ or ortuno@, I need an owner lg.

[ortuno, 2016-04-22 21:11:32]

lgtm

[stevenjb, 2016-04-22 21:13:27]

The CQ bit was checked by stevenjb@chromium.org

[commit-bot: I haz the power, 2016-04-22 21:14:51]

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

[commit-bot: I haz the power, 2016-04-22 21:20:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 21:21:33]

Description was changed from

==========
Bluez: Move released profiles to a separate map.

This addresses an edge case where released profiles may not have been
removed before Shutdown. See issue for details.

BUG=604710
==========

to

==========
Bluez: Move released profiles to a separate map.

This addresses an edge case where released profiles may not have been
removed before Shutdown. See issue for details.

BUG=604710

Committed: https://crrev.com/3bfed672ec31b7f51390da4c6ba260942ea8df25
Cr-Commit-Position: refs/heads/master@{#389242}
==========

[commit-bot: I haz the power, 2016-04-22 21:21:34]

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