Use FILE thread for disk operations in TraceSubscriberStdio

content/browser/trace_message_filter.cc

Index: content/browser/trace_message_filter.cc
===================================================================
--- content/browser/trace_message_filter.cc	(revision 120962)
+++ content/browser/trace_message_filter.cc	(working copy)
@@ -96,8 +96,9 @@
 }
 
 void TraceMessageFilter::OnTraceDataCollected(const std::string& data) {
-  TraceController::GetInstance()->OnTraceDataCollected(
-      make_scoped_refptr(new base::debug::TraceLog::RefCountedString(data)));
+  scoped_refptr<base::RefCountedString> string(new base::RefCountedString());

[joth, 2012/02/08 13:22:20]

avoid calling is 'string' as that's already overloaded.

[Iain Merrick, 2012/02/08 14:11:24]

Done.

+  string->data() = data;

[Iain Merrick, 2012/02/08 12:07:05]

This is a copy, not sure if there's a way to avoid it. Can we take ownership of
strings from IPC messages?

[joth, 2012/02/08 13:22:20]

At least you're not adding a new copy ..
I think it could work if you make data a non-const ref. (opps!) because then you
could use RefCountedString::TakeString()

[Iain Merrick, 2012/02/08 14:11:24]

But I only have a const IPC::Message, so yeah, sadly not possible. Makes sense
as this is a MessageFilter, and the same Message will be passed to other
filters.

+  TraceController::GetInstance()->OnTraceDataCollected(string);
 }
 
 void TraceMessageFilter::OnTraceBufferFull() {