GitLab

... off of content::RenderThreadImpl.

This CL changes the ownership of blink::PeerConnectionDependencyFactory
as per section 5.1.1 in the design document [1].

[1] https://docs.google.com/document/d/1AJKVA5U4nDkyDB9p4ROrggWXadCxyy-grKaE9KS5vOU/

Quoting the document:

  "
  PeerConnectionDependencyFactory is currently owned by RenderThreadImpl in content,
  which makes it a single object per renderer process. This ownership must be moved to
  a suitable place in Blink. A possibility is to keep it as a global object (using
  DEFINE_STATIC_LOCAL).
  The disadvantage of this approach is that PCDF would never be destroyed, whereas
  it is currently destroyed by the RenderThreadImpl destructor, but this is acceptable.
  "

This move will allow a set of blink::Platform methods to be removed
(follow up CL).

BUG=787254
R=guidou@chromium.org, haraken@chromium.org

Change-Id: Ie79f13cfd90c56fcbed50142802b0fcda480c43e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807657
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697750}