Revert "Revert "Const-ify RefCountedThreadSafe::AddRef and Release.""

chrome/browser/browser_thread_unittest.cc

Index: chrome/browser/browser_thread_unittest.cc
diff --git a/chrome/browser/browser_thread_unittest.cc b/chrome/browser/browser_thread_unittest.cc
index d2c19d8b921b94a7a8b9c2d8af91a2dc0c140d67..948709acc6d1b5ed67973fb99e1e4bcc26ddaae6 100644
--- a/chrome/browser/browser_thread_unittest.cc
+++ b/chrome/browser/browser_thread_unittest.cc
@@ -11,7 +11,7 @@
 
 class BrowserThreadTest : public testing::Test {
  public:
-  void Release() {
+  void Release() const {
     CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
     loop_.PostTask(FROM_HERE, new MessageLoop::QuitTask);
   }
@@ -77,7 +77,9 @@ class BrowserThreadTest : public testing::Test {
  private:
   scoped_ptr<BrowserThread> ui_thread_;
   scoped_ptr<BrowserThread> file_thread_;
-  MessageLoop loop_;
+  // It's kind of ugly to make this mutable - solely so we can post the Quit
+  // Task from Release(). This should be fixed.
+  mutable MessageLoop loop_;
 };
 
 TEST_F(BrowserThreadTest, PostTask) {