Remove unused gpu message fields.

Remove unused gpu message fields.


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

[no sievers, 2012-12-12 23:17:15]

All Mac-specific fields as well as width/height in the SurfaceNew seem unused
nowadays. But maybe the trybots will prove me wrong.

[no sievers, 2012-12-12 23:55:32]

On 2012/12/12 23:17:15, Daniel Sievers wrote:
> All Mac-specific fields as well as width/height in the SurfaceNew seem unused
> nowadays. But maybe the trybots will prove me wrong.

As the next step I want to get rid of the SurfaceNew message entirely and send
the mailbox name with the Swap and ACK.

That way I can make the logic in RWHV compatible with the way the renderer will
send frames to the mailbox, also just with a single message.

[piman, 2012-12-13 00:00:38]

LGTM
You'll need a security person to look at the message changes.

[no sievers, 2012-12-13 00:04:58]

+cevans for security review of gpu_messages.h

[Chris Evans, 2012-12-13 00:11:03]

On 2012/12/13 00:04:58, Daniel Sievers wrote:
> +cevans for security review of gpu_messages.h

LGTM!

[jonathan.backer, 2012-12-13 14:21:58]

https://codereview.chromium.org/11519045/diff/9002/content/browser/renderer_h...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

https://codereview.chromium.org/11519045/diff/9002/content/browser/renderer_h...
content/browser/renderer_host/render_widget_host_view_aura.cc:985: gfx::Size(0,
0), device_scale_factor_,
Do we still need the size? A quick code search suggests otherwise.

[no sievers, 2012-12-13 21:44:25]

On 2012/12/13 14:21:58, jonathan.backer wrote:
>
https://codereview.chromium.org/11519045/diff/9002/content/browser/renderer_h...
> File content/browser/renderer_host/render_widget_host_view_aura.cc (right):
> 
>
https://codereview.chromium.org/11519045/diff/9002/content/browser/renderer_h...
> content/browser/renderer_host/render_widget_host_view_aura.cc:985:
gfx::Size(0,
> 0), device_scale_factor_,
> Do we still need the size? A quick code search suggests otherwise.

Ok, I'll try to remove it with my next patch where I'm trying to get rid of
AcceleratedSurfaceNew entirely.

[commit-bot: I haz the power, 2012-12-13 21:45:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sievers@chromium.org/11519045/14001

[commit-bot: I haz the power, 2012-12-14 01:11:20]

Change committed as 173029