Delete implementation of ChromeRenderViewObserver::DidHandleTouchEvent

Delete implementation of ChromeRenderViewObserver::DidHandleTouchEvent

The code was crashing, the implementation was wrong anyway, and the result
was unused! This is a safe fix for M25 that just deletes the whole method
implementation. Not deleting the method because Ananta already has a change
in progress that reimplements it the correct way and makes use of it.

BUG=162433
TBR=darin


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

[Rick Byers, 2012-12-18 19:22:57]

This seems fine to me, modulo the comment below.  This code was originally added
for the CrOS onscreen keyboard, which isn't currently functional.  But we may be
trying to get it working again sometime soon.  Anata, can you point to the
related bug/CL you're working on so we can make sure CrOS and Windows are
aligned here?

Thanks for fixing the crash Dominic!
  Rick

https://codereview.chromium.org/11606011/diff/3001/chrome/renderer/chrome_ren...
File chrome/renderer/chrome_render_view_observer.cc (left):

https://codereview.chromium.org/11606011/diff/3001/chrome/renderer/chrome_ren...
chrome/renderer/chrome_render_view_observer.cc:690: render_view()->Send(new
ChromeViewHostMsg_FocusedEditableNodeTouched(
If you're removing all senders of this message, then you should probably remove
the other code using it too to be consistent (eg. remove
ChromeRenderViewHostObserver::OnFocusedEditableNodeTouched, and remove the IPC
message).

[dmazzoni, 2012-12-19 22:14:51]

FYI, Ananta just landed a change that sends
ChromeViewHostMsg_FocusedEditableNodeTouched from a different method,
DidHandleGestureEvent, so we should be good.

See http://crbug.com/166516 for more info.

Please cq+ when reviewing if possible.

[Rick Byers, 2012-12-19 22:20:15]

Thanks, lgtm
You still need OWNERS though

[dmazzoni, 2012-12-19 22:22:22]

On 2012/12/19 22:20:15, Rick Byers wrote:
> Thanks, lgtm
> You still need OWNERS though

Thanks - I'm going to cq+ with tbr=darin since he was already following this bug
and I didn't hear any objections. I'd like to make sure we can merge this before
the holidays.

[commit-bot: I haz the power, 2012-12-19 22:22:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmazzoni@chromium.org/11606011/7001

[commit-bot: I haz the power, 2012-12-20 00:46:59]

Change committed as 174061