Remove FINAL_STATUS_HTML5_MEDIA and related messages.

Remove FINAL_STATUS_HTML5_MEDIA and related messages.

They've been unused since r113742 when audio/video became deferred on
prerender. Also update a comment to account for media no longer being cancelled
that way.

Also remove some other dead code.

BUG=none
TEST=compiles, PrerenderBrowserTest.*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224766

[davidben, 2013-09-18 18:11:18]

Noticed this as I was reading code.

jschuh: for chrome/common/render_messages.h (I'm just deleting an IPC)

[mmenke, 2013-09-18 18:16:28]

On 2013/09/18 18:11:18, David Benjamin wrote:
> Noticed this as I was reading code.
> 
> jschuh: for chrome/common/render_messages.h (I'm just deleting an IPC)

LGTM!  You're finding so much stuff, it's a bit embarrassing.  I had nothing to
do with this one, at least.  :)

[davidben, 2013-09-19 18:55:12]

jschuh: ping

Trimmed off a little bit more code. I suspect we can get rid of
PrerenderRenderViewHostObserver altogether and move its OnMessageReceived (which
now exists only to service printing) back into PrerenderContents via
WebContentsObserver. Though I'm pretty sure the window.print logic incorrectly
cancels calls during a use/print race anyway...

Anyway, one thing at a time... :-)

[mmenke, 2013-09-19 19:52:33]

https://codereview.chromium.org/23484053/diff/9001/chrome/browser/prerender/p...
File chrome/browser/prerender/prerender_final_status.h (right):

https://codereview.chromium.org/23484053/diff/9001/chrome/browser/prerender/p...
chrome/browser/prerender/prerender_final_status.h:35:
FINAL_STATUS_SOURCE_RENDER_VIEW_CLOSED = 20,
Doesn't look like this isn't used any more, either.  Believe it's been replaced
by the abandoned logic.

[davidben, 2013-09-19 20:04:12]

https://codereview.chromium.org/23484053/diff/9001/chrome/browser/prerender/p...
File chrome/browser/prerender/prerender_final_status.h (right):

https://codereview.chromium.org/23484053/diff/9001/chrome/browser/prerender/p...
chrome/browser/prerender/prerender_final_status.h:35:
FINAL_STATUS_SOURCE_RENDER_VIEW_CLOSED = 20,
On 2013/09/19 19:52:33, mmenke wrote:
> Doesn't look like this isn't used any more, either.  Believe it's been
replaced
> by the abandoned logic.

Done.

[jschuh, 2013-09-19 21:08:18]

ipc security lgtm: message removal :)

[mmenke, 2013-09-19 21:10:10]

On 2013/09/19 21:08:18, Justin Schuh wrote:
> ipc security lgtm: message removal :)

Still LGTM to me (With the other message removed).

[commit-bot: I haz the power, 2013-09-19 21:21:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davidben@chromium.org/23484053/21001

[commit-bot: I haz the power, 2013-09-20 02:09:13]

Retried try job too often on win_rel for step(s) sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-09-20 18:35:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davidben@chromium.org/23484053/21001

[commit-bot: I haz the power, 2013-09-20 18:48:50]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[commit-bot: I haz the power, 2013-09-20 19:40:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davidben@chromium.org/23484053/21001

[commit-bot: I haz the power, 2013-09-20 22:15:02]

Retried try job too often on win_rel for step(s) sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-09-23 15:32:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davidben@chromium.org/23484053/21001

[commit-bot: I haz the power, 2013-09-23 19:26:23]

Change committed as 224766