Use enum instead of string in MessageLoopFactory::GetMessageLoop*

content/renderer/media/media_stream_impl.cc

Index: content/renderer/media/media_stream_impl.cc
diff --git a/content/renderer/media/media_stream_impl.cc b/content/renderer/media/media_stream_impl.cc
index b68d34d2a4ecc375730cbc1efdef9117d22b74c4..4fe59d849d3ed2f11e3c04c8fa6d11973e989acc 100644
--- a/content/renderer/media/media_stream_impl.cc
+++ b/content/renderer/media/media_stream_impl.cc
@@ -486,7 +486,8 @@ scoped_refptr<media::VideoDecoder> MediaStreamImpl::CreateLocalVideoDecoder(
            << video_session_id;
 
   return new CaptureVideoDecoder(
-      message_loop_factory->GetMessageLoopProxy("CaptureVideoDecoderThread"),
+      message_loop_factory->GetMessageLoopProxy(
+          media::MessageLoopFactory::VIDEO_DECODER),
       video_session_id,
       vc_manager_.get(),
       capability);
@@ -502,7 +503,8 @@ scoped_refptr<media::VideoDecoder> MediaStreamImpl::CreateRemoteVideoDecoder(
            << stream->label();
 
   return new RTCVideoDecoder(
-      message_loop_factory->GetMessageLoopProxy("RtcVideoDecoderThread"),
+      message_loop_factory->GetMessageLoopProxy(
+          media::MessageLoopFactory::VIDEO_DECODER),

[Ami GONE FROM CHROMIUM, 2012/08/09 17:19:39]

This change makes it so that a single thread is used for this and
CreateLocalVideoDecoder() above, where before 2 were used.
I'm not sure if that's a problem (i.e. can both methods ever be called with the
same factory, i.e. the same media element?)

[xhwang, 2012/08/10 01:04:28]

As per scherkus@, either local or remote video decoder is used in one media
element, not both. So we should be fine here.

       base::MessageLoopProxy::current(),
       stream->video_tracks()->at(0));
 }