Move bulk of event_with_latency_info into common.

Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802
Committed: https://crrev.com/d94b93a1c0ae2a1d49d78834284d54ad114ae38e
Cr-Commit-Position: refs/heads/master@{#372462}

[dtapuska, 2016-01-25 21:02:31]

Description was changed from

==========
Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802
==========

to

==========
Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802
==========

[dtapuska, 2016-01-25 21:02:32]

dtapuska@chromium.org changed reviewers:
+ tdresser@chromium.org

[tdresser, 2016-01-26 13:40:22]

https://codereview.chromium.org/1630983002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/event_with_latency_info.h (right):

https://codereview.chromium.org/1630983002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/event_with_latency_info.h:14:
NativeWebKeyboardEventWithLatencyInfo;
Why can't this move as well?

It's a bit confusing to have these spread out.

[dtapuska, 2016-01-26 14:16:16]

https://codereview.chromium.org/1630983002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/event_with_latency_info.h (right):

https://codereview.chromium.org/1630983002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/event_with_latency_info.h:14:
NativeWebKeyboardEventWithLatencyInfo;
On 2016/01/26 13:40:22, tdresser wrote:
> Why can't this move as well?
> 
> It's a bit confusing to have these spread out.

This is defined in content/public/browser/

I didn't want to introduce a dependency from common->browser. Because presumably
there are dependencies from browser->common.

I'm open to other suggestions.

[tdresser, 2016-01-26 14:20:09]

LGTM

https://codereview.chromium.org/1630983002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/event_with_latency_info.h (right):

https://codereview.chromium.org/1630983002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/event_with_latency_info.h:14:
NativeWebKeyboardEventWithLatencyInfo;
On 2016/01/26 14:16:16, dtapuska wrote:
> On 2016/01/26 13:40:22, tdresser wrote:
> > Why can't this move as well?
> > 
> > It's a bit confusing to have these spread out.
> 
> This is defined in content/public/browser/
> 
> I didn't want to introduce a dependency from common->browser. Because
presumably
> there are dependencies from browser->common.
> 
> I'm open to other suggestions.

Acknowledged.

[dtapuska, 2016-01-29 22:03:22]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-01-29 22:03:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdresser@chromium.org
Link to the patchset: https://codereview.chromium.org/1630983002/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-01-29 22:05:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1630983002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1630983002/20001

[commit-bot: I haz the power, 2016-01-29 23:23:35]

Description was changed from

==========
Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802
==========

to

==========
Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802
==========

[commit-bot: I haz the power, 2016-01-29 23:23:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-29 23:25:02]

Description was changed from

==========
Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802
==========

to

==========
Move bulk of event_with_latency_info into common.

Move the definition of EventWithLatencyInfo into common for code
reuse with a later change.

BUG=489802

Committed: https://crrev.com/d94b93a1c0ae2a1d49d78834284d54ad114ae38e
Cr-Commit-Position: refs/heads/master@{#372462}
==========

[commit-bot: I haz the power, 2016-01-29 23:25:03]

Patchset 2 (id:??) landed as
https://crrev.com/d94b93a1c0ae2a1d49d78834284d54ad114ae38e
Cr-Commit-Position: refs/heads/master@{#372462}