content: Make ifdefs match with where this is used

content: Move touchui's RenderWidgetHostView::GetDefaultScreenInfo() into a place to make things link

(Fixes trybot linkage on linux_touch)

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

[Elliot Glaysher, 2011-08-25 21:36:31]

This ifdef says mac, but in the header it says posix and in the RWH where it's
used it says posix.

[jam, 2011-08-25 21:42:27]

this was moved over from render_widget_host_view_mac.mm in
http://codereview.chromium.org/7746008 to fix a link error on mac.

[Elliot Glaysher, 2011-08-25 21:59:25]

On 2011/08/25 21:42:27, John Abd-El-Malek wrote:
> this was moved over from render_widget_host_view_mac.mm in
> http://codereview.chromium.org/7746008 to fix a link error on mac.

Closing then. I don't know why it's failing on the trybots

[jam, 2011-08-25 23:13:54]

http://codereview.chromium.org/7753013/diff/4001/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view.cc (right):

http://codereview.chromium.org/7753013/diff/4001/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view.cc:25: // TODO(erg): move
this to render_widget_host_view_views_gtk.cc when if it
this would be the 3d copy of this code :)

see the comment in render_widget_host_view_views_gtk.cc:
// TODO(oshima): This is a copy from RenderWidgetHostViewGtk. Replace this

If we're gonna reuse this, then I think that function should go into a gtk file
in content. Then we can also one cc file that's included for both gtk + touch
builds that has the :GetDefaultScreenInfo() function, since now it's duplicated
with the one in  render_widget_host_view_gtk.cc.

[jam, 2011-08-26 00:04:51]

great cleanup, thanks

lgtm

[commit-bot: I haz the power, 2011-08-26 03:00:34]

Change committed as 98389