Migrate webkit/renderer/media/ to content/renderer/media/.

chrome/renderer/prerender/DEPS

Index: chrome/renderer/prerender/DEPS
diff --git a/chrome/renderer/prerender/DEPS b/chrome/renderer/prerender/DEPS
new file mode 100644
index 0000000000000000000000000000000000000000..975daebca94f245a67b185cd80ee52f37f318fe6
--- /dev/null
+++ b/chrome/renderer/prerender/DEPS
@@ -0,0 +1,5 @@
+include_rules = [
+  # TODO(shishir): We should not be depending on content internals objects.
+  # XXX FILE BUG XXX
+  "+content/renderer/media/webmediaplayer_impl.h",

[jam, 2013/06/27 21:22:51]

can't do this.. chrome can't include anything internal from content even
temporarily. please see the content api wiki
http://www.chromium.org/developers/content-module/content-api

since we don't have implementation inheritance in the content api, how we
usually solve this is moving to containment instead

[scherkus (not reviewing), 2013/06/27 21:29:31]

Can you give an example?

Do you mean wrapping a WebKit::WebMediaPlayer object + reimplementing the
interface + forwarding all the methods?

As an aside ... do you know why this scrutiny wasn't applied to anything under
src/webkit/ during content-ification of chrome? Was it out of scope?

[jam, 2013/06/28 00:31:58]

what i mean is that the prerendering code doesn't implement the interface, it
inherits from the implementation. we don't allow implementation inheritance in
the content api, since that would mean exposing internal implementation details
of content.

it's best to study that code to see exactly what prerendering needs to do. there
may be simpler ways of achieving this.
the chrome/content split was done for code in src/chrome. we didn't touch other
top level directories like wekbit, net etc.

+]