• Antonio Gomes's avatar
    Create WebMediaStreamRendererFactory instances from blink · 3b3d2a16
    Antonio Gomes authored
    ... instead of content/renderer/media.
    
    Today, content/renderer/media/media_factory.cc calls out to
    blink::CreateWebMediaStreamRendererFactory() (from
    t_p/blink/public/web/modules/mediastream/web_media_stream_renderer_factory.h)
    and passes the object to Blink.
    
    This CL moves this call within WebMediaPlayerMS's ctor, in webmediaplayer_ms.cc
    (blink). For tests, it adds an override method.
    
    This will allow:
    - merged of WebMediaStreamRendererFactory and
    MediaStreamRendererFactoryImpl classes.
    - removal of web_media_stream_renderer_factory.h.
    
    BUG=704136
    R=guidou@chromium.org, haraken@chromium.org
    
    Change-Id: I368f9063cfa9a361b603d35f9c9b30d31b488837
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302711
    Auto-Submit: Antonio Gomes <tonikitoo@igalia.com>
    Commit-Queue: Kentaro Hara <haraken@chromium.org>
    Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
    Reviewed-by: default avatarGuido Urdaneta <guidou@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#789308}
    3b3d2a16
webmediaplayer_ms.cc 47 KB