Fix heapcheck errors

content/browser/streams/stream_context.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/streams/stream_context.h"
 
 #include "base/bind.h"
 #include "content/browser/streams/stream_registry.h"
 #include "content/public/browser/browser_context.h"
 
@@ skipping 23 matching lines @@
   return UserDataAdapter<StreamContext>::Get(context, kStreamContextKeyName);
 }
 
 void StreamContext::InitializeOnIOThread() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   registry_.reset(new StreamRegistry());
 }
 
 StreamContext::~StreamContext() {}
 
+void StreamContext::DeleteOnCorrectThread() const {

[darin (slow to review), 2013/03/11 20:26:41]

I'm really not a big fan of test-only code like this.  It runs the risk of being
used in production, bloats production code for the sake of testing, and just
generally makes the code more complex.  Did you consider adding an IO thread to
the tests?

+  // In many tests, there isn't a valid IO thread.  In that case, just delete on
+  // the current thread.
+  if (BrowserThread::IsMessageLoopValid(BrowserThread::IO) &&
+      !BrowserThread::CurrentlyOn(BrowserThread::IO)) {
+    BrowserThread::DeleteSoon(BrowserThread::IO, FROM_HERE, this);
+    return;
+  }
+  delete this;
+}
+
 }  // namespace content