ui: Fix removing a number from the SequentialIDGenerator.

ui: Fix removing a number from the SequentialIDGenerator.

Make sure when a number is released, the |min_available_slot_| is updated accordingly,
so that subsequent allocations are set correctly to the lowest available ID.

BUG=292929
R=sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223517

[sadrul, 2013-09-16 22:07:21]

https://codereview.chromium.org/23654045/diff/1/ui/base/sequential_id_generat...
File ui/base/sequential_id_generator.cc (right):

https://codereview.chromium.org/23654045/diff/1/ui/base/sequential_id_generat...
ui/base/sequential_id_generator.cc:63: uint32 id = number_to_id_[number];
Should I just call ReleaseGeneratedID(number_to_id_[number]); here instead?

[sky, 2013-09-16 23:49:26]

LGTM

https://codereview.chromium.org/23654045/diff/5001/ui/base/sequential_id_gene...
File ui/base/sequential_id_generator.cc (right):

https://codereview.chromium.org/23654045/diff/5001/ui/base/sequential_id_gene...
ui/base/sequential_id_generator.cc:64: if (id < min_available_id_) {
Refactor 54-57 and 64-67 to a common method?

[sadrul, 2013-09-17 02:26:38]

https://codereview.chromium.org/23654045/diff/5001/ui/base/sequential_id_gene...
File ui/base/sequential_id_generator.cc (right):

https://codereview.chromium.org/23654045/diff/5001/ui/base/sequential_id_gene...
ui/base/sequential_id_generator.cc:64: if (id < min_available_id_) {
On 2013/09/16 23:49:27, sky wrote:
> Refactor 54-57 and 64-67 to a common method?

Done.

[sadrul, 2013-09-17 02:27:53]

Committed patchset #3 manually as r223517 (presubmit successful).