commit | a5856e431d7be09dbbde0ab274fdddca868bb269 | [log] [tgz] |
---|---|---|
author | Alex Deymo <deymo@chromium.org> | Mon Mar 31 18:01:36 2014 -0700 |
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | Tue Apr 01 21:36:39 2014 +0000 |
tree | 37534fa6b5b42b0e776a1ad1952f3c1c379583a3 | |
parent | 0bc10fafd1bb55186fe1f87805967d157e6d18ad [diff] |
PolicyManager: Call all the ValueChanged() on the same main loop call. This patch calls the ValueChanged() method on all the observers from the same main loop calling, fixing a possible free after use if the observer gets removed and deleted between the NotifyValueChanged() call and the ValueChanged() calls. BUG=chromium:355132 TEST=Unittest added. Change-Id: I6365c3b86fe0b85502bfa81bbd18a86c55caa009 Reviewed-on: https://chromium-review.googlesource.com/192526 Reviewed-by: Alex Deymo <deymo@chromium.org> Commit-Queue: Alex Deymo <deymo@chromium.org> Tested-by: Alex Deymo <deymo@chromium.org>