The base unittests were succeeding for the isolated tests, but failed when all tests were run because some tests run without a message loop.  The  observer_list_threadsafe allowed AddObserver to run with null message loops.  But, if you call Notify after doing so, then the notify call would attempt to use the registered "null" ptr to a message loop.
The base unittests were succeeding for the isolated
tests, but failed when all tests were run because
some tests run without a message loop.  The 
observer_list_threadsafe allowed AddObserver to run
with null message loops.  But, if you call Notify
after doing so, then the notify call would attempt
to use the registered "null" ptr to a message loop.

This crashed, of course.

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

git-svn-id: dd90618784b6a4b323ea0c23a071cb1c9e6f2ac7@4459 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser