commit cbec5fb0aa42c03df1a205f0aac6fb8d4a587be4
author akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Sun Jun 05 16:07:12 2011 +0900
committer Qijiang Fan <fqj@google.com> Thu Jun 04 11:29:03 2020 +0900
tree aec8d04795d8949c4910acec964f1c23cd00eee8
parent 3b365faa4ca77f2116fe6c086673da284226ec13

Fix bug in ObserverListThreadsafe::RemoveObserver

Like ObserverList, ObserverListThreadSafe::RemoveObserver should do
nothing if the observer isn't in the list.  Instead, it causes weird
bugs if the list has 0 or 1 existing elements.

Add comments clarifying semantics of AddObserver/RemoveObserver.

Change AddObserver to do nothing in release mode if it tries to add an existing observer.

BUG=84922
TEST=


Review URL: http://codereview.chromium.org/7024037

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87944 0039d316-1c4b-4281-b951-d872f2087c98


CrOS-Libchrome-Original-Commit: 631739f16f16c09b1a08d219c5bcb0c776dddb42