Associate an id with the output surface to handle lost contexts

Associate an id with the output surface to handle lost contexts

This lets us cull acks that were meant for a previous output surface, so that we
don't try to return/release resources that have been destroyed already.

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

[piman, 2013-07-16 04:23:05]

sievers: content/browser/renderer_host/render_widget_host_view_android.*,
content/renderer/gpu/mailbox_output_surface.*
alexst: content/browser/browser_plugin/browser_plugin_guest.*,
content/browser/renderer_host/render_widget_host_view_guest.*,
content/renderer/browser_plugin/*
danakj: the rest
palmer: IPC

[piman, 2013-07-16 04:24:30]

https://codereview.chromium.org/19331002/diff/1/content/renderer/render_widge...
File content/renderer/render_widget.cc (right):

https://codereview.chromium.org/19331002/diff/1/content/renderer/render_widge...
content/renderer/render_widget.cc:641: return
factory->CreateOutputSurface(routing_id());
note: I believe we don't need to pass the output_surface_id here, because we're
in process and everything is synchronous anyway. If we do, let me know.

[alexst (slow to review), 2013-07-16 13:33:03]

browser plugin lgtm

[no sievers, 2013-07-16 16:46:26]

On 2013/07/16 04:23:05, piman wrote:
> sievers: content/browser/renderer_host/render_widget_host_view_android.*,

lgtm

> content/renderer/gpu/mailbox_output_surface.*
> alexst: content/browser/browser_plugin/browser_plugin_guest.*,
> content/browser/renderer_host/render_widget_host_view_guest.*,
> content/renderer/browser_plugin/*
> danakj: the rest
> palmer: IPC

[palmer, 2013-07-16 19:28:19]

IPC security LGTM

[danakj, 2013-07-17 21:07:28]

LGTM

[commit-bot: I haz the power, 2013-07-18 19:10:33]

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

[commit-bot: I haz the power, 2013-07-18 23:02:32]

Change committed as 212470