Move |DidOverscrollParams| from |content::| to "ui/events/blink"

Move |DidOverscrollParams| from |content::| to "ui/events/blink"

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
Committed: https://crrev.com/a8ba91fcbea546ad0922f99aa89fc19670c3b129
Cr-Commit-Position: refs/heads/master@{#412344}

[chongz, 2016-08-12 19:37:18]

The CQ bit was checked by chongz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 19:37:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 19:41:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-12 19:41:55]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[chongz, 2016-08-12 20:02:20]

The CQ bit was checked by chongz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 20:03:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 20:24:29]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-12 20:24:30]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[chongz, 2016-08-12 21:31:13]

The CQ bit was checked by chongz@chromium.org to run a CQ dry run

[chongz, 2016-08-12 21:31:45]

Patchset #1 (id:1) has been deleted

[chongz, 2016-08-12 21:31:53]

Patchset #1 (id:20001) has been deleted

[chongz, 2016-08-12 21:32:02]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2016-08-12 21:32:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chongz, 2016-08-12 21:32:20]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2016-08-12 22:42:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-12 22:42:04]

Dry run: This issue passed the CQ dry run.

[chongz, 2016-08-14 02:40:30]

Description was changed from

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy|.

BUG=637393
==========

to

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy|.

Added dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

[chongz, 2016-08-14 02:40:44]

Description was changed from

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy|.

Added dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

to

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy|.

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

[chongz, 2016-08-15 13:11:41]

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

[chongz, 2016-08-15 13:11:42]

dtapuska@ PTAL, thanks!

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
File ui/events/blink/did_overscroll_params.cc (left):

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
ui/events/blink/did_overscroll_params.cc:11: }
Wrong file for diff?

[dtapuska, 2016-08-15 18:50:48]

lgtm

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
File ui/events/blink/did_overscroll_params.h (right):

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
ui/events/blink/did_overscroll_params.h:13: struct DidOverscrollParams {
Do you need a export this at all?

[chongz, 2016-08-15 19:08:43]

Description was changed from

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy|.

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

to

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

[chongz, 2016-08-15 19:09:12]

chongz@chromium.org changed reviewers:
+ jam@chromium.org

[chongz, 2016-08-15 19:09:13]

jam@ PTAL, thanks!

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
File ui/events/blink/did_overscroll_params.h (right):

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
ui/events/blink/did_overscroll_params.h:13: struct DidOverscrollParams {
On 2016/08/15 18:50:47, dtapuska wrote:
> Do you need a export this at all?

I'm not sure but it seems working and no bot is complaining about it...

[chongz, 2016-08-16 14:59:34]

Description was changed from

==========
Move |DidOverscrollParams| from |content::| to |ui::|

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

to

==========
Move |DidOverscrollParams| from |content::| to "ui/events/blink"

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

[chongz, 2016-08-16 15:01:00]

chongz@chromium.org changed reviewers:
+ nick@chromium.org

[chongz, 2016-08-16 15:01:00]

ncarter@ PTAL at this CL, thanks!
(seems jam@ is OOO.)

[ncarter (slow), 2016-08-16 19:26:42]

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
File ui/events/blink/did_overscroll_params.cc (left):

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
ui/events/blink/did_overscroll_params.cc:11: }
On 2016/08/15 13:11:42, chongz wrote:
> Wrong file for diff?

This is unavoidable sometimes, especially with small files. There's nothing you
can do really.

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
File ui/events/blink/did_overscroll_params.h (right):

https://codereview.chromium.org/2240273002/diff/60001/ui/events/blink/did_ove...
ui/events/blink/did_overscroll_params.h:13: struct DidOverscrollParams {
On 2016/08/15 19:09:13, chongz wrote:
> On 2016/08/15 18:50:47, dtapuska wrote:
> > Do you need a export this at all?
> 
> I'm not sure but it seems working and no bot is complaining about it...

It's not required because ui/events/blink exports things as a source_set,
meaning (if I understand correctly) that the obj files are linked separately
into each dependent component. So we get a copy of ~DidOverscrollParams linked
into each dll.

Why does ui/events/blink use a source set and not a component? No idea.

[ncarter (slow), 2016-08-16 19:26:56]

content lgtm

[chongz, 2016-08-16 19:37:33]

chongz@chromium.org changed reviewers:
+ kenrb@chromium.org

[chongz, 2016-08-16 19:37:34]

kenrb@ PTAL at IPC, thanks!

[kenrb, 2016-08-16 20:07:18]

lgtm

[chongz, 2016-08-16 20:44:39]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2016-08-16 20:45:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-16 21:39:32]

Description was changed from

==========
Move |DidOverscrollParams| from |content::| to "ui/events/blink"

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

to

==========
Move |DidOverscrollParams| from |content::| to "ui/events/blink"

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

[commit-bot: I haz the power, 2016-08-16 21:39:34]

Committed patchset #1 (id:60001)

[commit-bot: I haz the power, 2016-08-16 21:42:07]

Description was changed from

==========
Move |DidOverscrollParams| from |content::| to "ui/events/blink"

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393
==========

to

==========
Move |DidOverscrollParams| from |content::| to "ui/events/blink"

This CL moves |DidOverscrollParams| from |content::| to |ui::|, so it
can be accessed by |InputHandlerProxy| in "ui/events/blink".

Adds dependency:
  1. content/common : ui/events/blink
  2. content/renderer/mus : ui/events/blink

BUG=637393

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

[commit-bot: I haz the power, 2016-08-16 21:42:08]

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