Remove ui/events/blink dependency on blink_minimal.

Remove ui/events/blink dependency on blink_minimal.

It only needs blink_headers. WebCoalescedInputEvent was not an enum/POD so it shouldn't have been used outside of renderer. Fixed by:
-moving ScopedWebInputEventWithLatencyInfo from content/common to content/renderer (its only user is there anyways)
-hiding blink::WebScopedInputEvent from the public API and creating another version in ui/events/blink/web_input_events_traits. that file already has a duplicate of all the per-specific-type operations 

Also update blink_headers to include some headers it was missing, and the OWNERS file which was stale after gyp->gn conversion.

BUG=248653
Review-Url: https://codereview.chromium.org/2683043004
Cr-Commit-Position: refs/heads/master@{#449631}
Committed: https://chromium.googlesource.com/chromium/src/+/013b74c8a427c4ec9a5465a6060f626db8cacfd5

[jam, 2017-02-09 20:51:54]

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

[commit-bot: I haz the power, 2017-02-09 20:52:29]

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

[jam, 2017-02-09 20:52:51]

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

[commit-bot: I haz the power, 2017-02-09 21:08:44]

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

[commit-bot: I haz the power, 2017-02-09 21:08:45]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dcheng, 2017-02-09 21:57:55]

Looks like the build is broken. I'll look again once it's green.

[jam, 2017-02-09 21:59:04]

ah, yeah building chrome was fine, but the test target wasn't. investigating.

[jam, 2017-02-09 23:58:21]

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

[commit-bot: I haz the power, 2017-02-09 23:59:14]

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

[jam, 2017-02-10 00:00:23]

Description was changed from

==========
Remove ui/events/blink dependency on blink_minimal.

It only needs blink_headers. Update blink_headers to include some headers it was
missing, and the OWNERS file which was stale after gyp->gn conversion.

BUG=248653
==========

to

==========
Remove ui/events/blink dependency on blink_minimal.

It only needs blink_headers. WebCoalescedInputEvent was not an enum/POD so it
shouldn't have been used outside of renderer. Fixed by:
-moving ScopedWebInputEventWithLatencyInfo from content/common to
content/renderer (its only user is there anyways)
-hiding blink::WebScopedInputEvent from the public API and creating another
version in ui/events/blink/web_input_events_traits. that file already has a
duplicate of all the per-specific-type operations 

Also update blink_headers to include some headers it was missing, and the OWNERS
file which was stale after gyp->gn conversion.

BUG=248653
==========

[jam, 2017-02-10 00:24:53]

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

[jam, 2017-02-10 00:25:51]

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

[jam, 2017-02-10 00:26:36]

Daniel: ptal this should be ready now (verified all builds on win & linux
locally)

Nico: ptal at third_party/WebKit/Source/platform

[commit-bot: I haz the power, 2017-02-10 00:26:36]

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, 2017-02-10 01:13:36]

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

[commit-bot: I haz the power, 2017-02-10 01:13:37]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[jam, 2017-02-10 01:55:14]

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

[commit-bot: I haz the power, 2017-02-10 01:55:37]

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, 2017-02-10 02:56:31]

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

[commit-bot: I haz the power, 2017-02-10 02:56:32]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[dcheng, 2017-02-10 10:17:26]

lgtm

https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
File content/renderer/input/scoped_web_input_event_with_latency_info.cc (right):

https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
content/renderer/input/scoped_web_input_event_with_latency_info.cc:14: :
event_(new blink::WebCoalescedInputEvent(*(event.get()))),
Nit: I think *event should suffice here.

[jam, 2017-02-10 15:48:59]

https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
File content/renderer/input/scoped_web_input_event_with_latency_info.cc (right):

https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
content/renderer/input/scoped_web_input_event_with_latency_info.cc:14: :
event_(new blink::WebCoalescedInputEvent(*(event.get()))),
On 2017/02/10 10:17:25, dcheng wrote:
> Nit: I think *event should suffice here.

Done.

[jam, 2017-02-10 15:53:31]

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

[commit-bot: I haz the power, 2017-02-10 15:53:51]

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

[Nico, 2017-02-10 15:59:25]

On 2017/02/10 15:48:59, jam wrote:
>
https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
> File content/renderer/input/scoped_web_input_event_with_latency_info.cc
(right):
> 
>
https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
> content/renderer/input/scoped_web_input_event_with_latency_info.cc:14: :
> event_(new blink::WebCoalescedInputEvent(*(event.get()))),
> On 2017/02/10 10:17:25, dcheng wrote:
> > Nit: I think *event should suffice here.
> 
> Done.

Lgtm

[jam, 2017-02-10 16:23:12]

The CQ bit was unchecked by jam@chromium.org

[jam, 2017-02-10 16:23:18]

The CQ bit was checked by jam@chromium.org

[jam, 2017-02-10 16:23:18]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2683043004/#ps80001
(title: "nit")

[commit-bot: I haz the power, 2017-02-10 16:23:52]

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

[jam, 2017-02-10 16:30:11]

Patchset #5 (id:80001) has been deleted

[jam, 2017-02-10 16:30:34]

The CQ bit was unchecked by jam@chromium.org

[jam, 2017-02-10 16:30:39]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-02-10 16:30:54]

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

[jam, 2017-02-10 16:30:59]

On 2017/02/10 15:48:59, jam wrote:
>
https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
> File content/renderer/input/scoped_web_input_event_with_latency_info.cc
(right):
> 
>
https://codereview.chromium.org/2683043004/diff/60001/content/renderer/input/...
> content/renderer/input/scoped_web_input_event_with_latency_info.cc:14: :
> event_(new blink::WebCoalescedInputEvent(*(event.get()))),
> On 2017/02/10 10:17:25, dcheng wrote:
> > Nit: I think *event should suffice here.
> 
> Done.

actually I'll do the nit in a followup, it'll be faster that way.

[commit-bot: I haz the power, 2017-02-10 16:36:14]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1486744239339410,
"parent_rev": "d3bbe536eab7b3fd2f22568dfb1650bf78732ba3", "commit_rev":
"013b74c8a427c4ec9a5465a6060f626db8cacfd5"}

[commit-bot: I haz the power, 2017-02-10 16:36:49]

Description was changed from

==========
Remove ui/events/blink dependency on blink_minimal.

It only needs blink_headers. WebCoalescedInputEvent was not an enum/POD so it
shouldn't have been used outside of renderer. Fixed by:
-moving ScopedWebInputEventWithLatencyInfo from content/common to
content/renderer (its only user is there anyways)
-hiding blink::WebScopedInputEvent from the public API and creating another
version in ui/events/blink/web_input_events_traits. that file already has a
duplicate of all the per-specific-type operations 

Also update blink_headers to include some headers it was missing, and the OWNERS
file which was stale after gyp->gn conversion.

BUG=248653
==========

to

==========
Remove ui/events/blink dependency on blink_minimal.

It only needs blink_headers. WebCoalescedInputEvent was not an enum/POD so it
shouldn't have been used outside of renderer. Fixed by:
-moving ScopedWebInputEventWithLatencyInfo from content/common to
content/renderer (its only user is there anyways)
-hiding blink::WebScopedInputEvent from the public API and creating another
version in ui/events/blink/web_input_events_traits. that file already has a
duplicate of all the per-specific-type operations 

Also update blink_headers to include some headers it was missing, and the OWNERS
file which was stale after gyp->gn conversion.

BUG=248653

Review-Url: https://codereview.chromium.org/2683043004
Cr-Commit-Position: refs/heads/master@{#449631}
Committed:
https://chromium.googlesource.com/chromium/src/+/013b74c8a427c4ec9a5465a6060f...
==========

[commit-bot: I haz the power, 2017-02-10 16:36:51]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/013b74c8a427c4ec9a5465a6060f...

[Reid Kleckner, 2017-02-14 02:14:44]

rnk@chromium.org changed reviewers:
+ rnk@chromium.org

[Reid Kleckner, 2017-02-14 02:14:45]

https://codereview.chromium.org/2683043004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebCoalescedInputEvent.h (right):

https://codereview.chromium.org/2683043004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebCoalescedInputEvent.h:32: struct
WebInputEventDeleter {
It looks like you need to add BLINK_PLATFORM_EXPORT here to satisfy clang:
https://build.chromium.org/p/chromium.fyi/builders/CrWinClang64%28dll%29/buil...

[dcheng, 2017-02-14 02:21:10]

https://codereview.chromium.org/2683043004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebCoalescedInputEvent.h (right):

https://codereview.chromium.org/2683043004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebCoalescedInputEvent.h:32: struct
WebInputEventDeleter {
On 2017/02/14 02:14:45, Reid Kleckner wrote:
> It looks like you need to add BLINK_PLATFORM_EXPORT here to satisfy clang:
>
https://build.chromium.org/p/chromium.fyi/builders/CrWinClang64%28dll%29/buil...

I think thakis already submitted a cl for this:
https://codereview.chromium.org/2694053003/