Merged base::ObserverListBase into base::ObserverList.
There was no need for a separate base class.
It didn't have any virtual method and thus didn't provide any customisation point.
Moved definition of NotificationType from base::ObserverListBase to namespace ::base itself and renamed it ObserverListPolicy.
This way, ObserverListPolicy doesn't depend on ObserverList's template parameters.
It makes it easier to refer to ObserverListPolicy enumeration constants.
Also made ObserverListPolicy a scoped enumeration, to prevent enumeration constants from polluting the ::base namespace.
In the end: less code, fewer types, to do exactly the same thing. There is no functional change.
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ic87acaad956145b222d156167292811016b8cdb0
TBR=cbentzel@chromium.org,hansberry@chromium.org,miu@chromium.org,rockot@chromium.org,sdefresne@chromium.org
Change-Id: Ic87acaad956145b222d156167292811016b8cdb0
Reviewed-on: https://chromium-review.googlesource.com/778599
Commit-Queue: François Degros <fdegros@chromium.org>
Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Reviewed-by: Yuzhu Shen <yzshen@chromium.org>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#518545}
CrOS-Libchrome-Original-Commit: d6e2d7ddfd6339b13d34b2d3ed9e27b96d9849a9
4 files changed