[DevTools] Move eye dropper functionality from protocol to embedder

[DevTools] Move eye dropper functionality from protocol to embedder

This feature does not make sense for remote debugging or automation,
which suggests it's an embedder feature.

BUG=none

Review-Url: https://codereview.chromium.org/2817593004
Cr-Commit-Position: refs/heads/master@{#464989}
Committed: https://chromium.googlesource.com/chromium/src/+/1137e62b9a4f027f2af1fa60a9dcd35e8f1043b8

[dgozman, 2017-04-14 16:32:33]

Description was changed from

==========
[DevTools] Move eye dropper functionality from protocol to embedder

This feature does not make sense for remote debugging, which suggests
it's an embedder feature.

BUG=none
==========

to

==========
[DevTools] Move eye dropper functionality from protocol to embedder

This feature does not make sense for remote debugging or automation,
which suggests it's an embedder feature.

BUG=none
==========

[dgozman, 2017-04-14 16:58:18]

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

[commit-bot: I haz the power, 2017-04-14 16:58:29]

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

[dgozman, 2017-04-14 17:02:05]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org, tsepez@chromium.org

[dgozman, 2017-04-14 17:02:05]

Hi, could you please take a look?

@Tom - devtools_embedder_message_dispatcher. I'm adding a method which lets
DevTools to show eyedropper (a tool to select color) over the page which changes
the cursor.
This is just a move of functionality, previously done on content/ level.

@Pavel - whole CL.

Thanks,
Dmitry

[Tom Sepez, 2017-04-14 18:35:53]

lgtm

[commit-bot: I haz the power, 2017-04-14 19:06:33]

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

[commit-bot: I haz the power, 2017-04-14 19:06:34]

Dry run: This issue passed the CQ dry run.

[pfeldman, 2017-04-17 17:42:00]

lgtm

[dgozman, 2017-04-17 17:49:45]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-04-17 17:50:11]

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-04-17 19:49:18]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492451385783080,
"parent_rev": "67c41de111691e8cfb173c6cb1f31ee1d24b5dfe", "commit_rev":
"1137e62b9a4f027f2af1fa60a9dcd35e8f1043b8"}

[commit-bot: I haz the power, 2017-04-17 19:49:31]

Description was changed from

==========
[DevTools] Move eye dropper functionality from protocol to embedder

This feature does not make sense for remote debugging or automation,
which suggests it's an embedder feature.

BUG=none
==========

to

==========
[DevTools] Move eye dropper functionality from protocol to embedder

This feature does not make sense for remote debugging or automation,
which suggests it's an embedder feature.

BUG=none

Review-Url: https://codereview.chromium.org/2817593004
Cr-Commit-Position: refs/heads/master@{#464989}
Committed:
https://chromium.googlesource.com/chromium/src/+/1137e62b9a4f027f2af1fa60a9dc...
==========

[commit-bot: I haz the power, 2017-04-17 19:49:32]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1137e62b9a4f027f2af1fa60a9dc...