Add a TODO from Jar.
Review URL: http://codereview.chromium.org/9262 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4475 0039d316-1c4b-4281-b951-d872f2087c98
This commit is contained in:
@ -64,6 +64,9 @@ class ObserverListThreadSafe :
|
|||||||
void AddObserver(ObserverType* obs) {
|
void AddObserver(ObserverType* obs) {
|
||||||
ObserverList<ObserverType>* list = NULL;
|
ObserverList<ObserverType>* list = NULL;
|
||||||
MessageLoop* loop = MessageLoop::current();
|
MessageLoop* loop = MessageLoop::current();
|
||||||
|
// TODO(mbelshe): Get rid of this check. Its needed right now because
|
||||||
|
// Time currently triggers usage of the ObserverList.
|
||||||
|
// And unittests use time without a MessageLoop.
|
||||||
if (!loop)
|
if (!loop)
|
||||||
return; // Some unittests may access this without a message loop.
|
return; // Some unittests may access this without a message loop.
|
||||||
{
|
{
|
||||||
|
Reference in New Issue
Block a user