Comment #13 on issue 16008 by [email protected]: Refactor  
media::Pipeline to support injected message loops
http://code.google.com/p/chromium/issues/detail?id=16008

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=20417

------------------------------------------------------------------------
r20417 | [email protected] | 2009-07-10 14:07:27 -0700 (Fri, 10 Jul  
2009) | 8 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/filter_host_impl.cc?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/filter_host_impl.h?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline.h?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline_impl.cc?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline_impl.h?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline_impl_unittest.cc?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/player/movie.cc?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/media/player/movie.h?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webmediaplayer_impl.cc?r1=20417&r2=20416
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webmediaplayer_impl.h?r1=20417&r2=20416

Implemented injected message loops for PipelineImpl.

For now both the player and WebMediaPlayerImpl create a thread and inject  
its message loop into the pipeline.  The end result is more-or-less the  
same as what we have today, but we could end up moving the pipeline onto  
the render thread.

BUG=16008
TEST=layout tests, media_unittests should pass

Review URL: http://codereview.chromium.org/155338
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to