Remove IsProtocolSupportedForMedia

Remove IsProtocolSupportedForMedia.

scherkus says:
The protocol whitelist (added in r26746) was originally required to prevent issuing requests to the resource loader bridge that it didn't support (i.e., data://). To complicate matters there were protocols such as chrome-extension:// that required the embedder to override. None of this is required anymore since we switched to using Chromium WebKit API's resource loader in r69429.

BUG=98755
TEST=Chrome, DRT, test_shell, test_shell_tests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111075

[piman, 2011-11-21 21:52:01]

scherkus: please review
tony: OWNERS

[Dirk Pranke, 2011-11-21 21:57:46]

why is it okay to remove this? (I'm curious as to what the discussion was that
ended up with this function no longer being needed).

[piman, 2011-11-21 22:26:24]

+darin for webkit/ OWNERS since tony looks like he's out.

[scherkus (not reviewing), 2011-11-21 22:26:41]

dpranke: added my explanation in CL description

LGTM but I think there are more things to remove (and consequently OWNERS to
add) under chrome/

http://codereview.chromium.org/8616008/diff/1/content/renderer/renderer_glue.cc
File content/renderer/renderer_glue.cc (left):

http://codereview.chromium.org/8616008/diff/1/content/renderer/renderer_glue....
content/renderer/renderer_glue.cc:170:
content::GetContentClient()->renderer()->IsProtocolSupportedForMedia(url);
should also remove methods in chrome/renderer/chrome_content_renderer_client.*

http://codereview.chromium.org/8616008/diff/1/webkit/glue/webkit_glue.h
File webkit/glue/webkit_glue.h (left):

http://codereview.chromium.org/8616008/diff/1/webkit/glue/webkit_glue.h#oldco...
webkit/glue/webkit_glue.h:207: 
nit: remove extra blank line

[darin (slow to review), 2011-11-21 22:41:07]

LGTM

[piman, 2011-11-21 23:13:02]

+jam for content/public OWNER

http://codereview.chromium.org/8616008/diff/1/content/renderer/renderer_glue.cc
File content/renderer/renderer_glue.cc (left):

http://codereview.chromium.org/8616008/diff/1/content/renderer/renderer_glue....
content/renderer/renderer_glue.cc:170:
content::GetContentClient()->renderer()->IsProtocolSupportedForMedia(url);
On 2011/11/21 22:26:41, scherkus wrote:
> should also remove methods in chrome/renderer/chrome_content_renderer_client.*

Done.

http://codereview.chromium.org/8616008/diff/1/webkit/glue/webkit_glue.h
File webkit/glue/webkit_glue.h (left):

http://codereview.chromium.org/8616008/diff/1/webkit/glue/webkit_glue.h#oldco...
webkit/glue/webkit_glue.h:207: 
On 2011/11/21 22:26:41, scherkus wrote:
> nit: remove extra blank line

Done.

[jam, 2011-11-21 23:56:10]

lgtm

[commit-bot: I haz the power, 2011-11-22 00:04:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/8616008/1018

[commit-bot: I haz the power, 2011-11-22 02:04:25]

Change committed as 111075