View should store a ViewTargeter instead of an EventTargeter

View should store a ViewTargeter instead of an EventTargeter

Change |targeter_| in View to be a ViewTargeter instead
of an EventTargeter. This will allow us to avoid casting
when calling into ViewTargeter-specific methods in
upcoming CLs.

BUG=388838
TEST=none

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

[tdanderson, 2014-06-26 21:06:52]

Sadrul, can you please take a look?

[sadrul, 2014-06-27 21:23:38]

lgtm

https://codereview.chromium.org/354063003/diff/20001/ui/views/view.h
File ui/views/view.h (right):

https://codereview.chromium.org/354063003/diff/20001/ui/views/view.h#newcode734
ui/views/view.h:734: ViewTargeter* GetViewTargeter() const;
Just ViewTargeter* targeter() { return targeter_.get(); }?

[tdanderson, 2014-06-27 22:01:05]

The CQ bit was checked by tdanderson@chromium.org

[tdanderson, 2014-06-27 22:01:13]

https://codereview.chromium.org/354063003/diff/20001/ui/views/view.h
File ui/views/view.h (right):

https://codereview.chromium.org/354063003/diff/20001/ui/views/view.h#newcode734
ui/views/view.h:734: ViewTargeter* GetViewTargeter() const;
On 2014/06/27 21:23:38, sadrul wrote:
> Just ViewTargeter* targeter() { return targeter_.get(); }?

Done.

[commit-bot: I haz the power, 2014-06-27 22:03:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tdanderson@chromium.org/354063003/40001

[commit-bot: I haz the power, 2014-06-28 01:46:17]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-06-28 05:46:15]

Change committed as 280501