Remove RenderFrameImpl::GetFocusedElement.

Remove RenderFrameImpl::GetFocusedElement.

The concept of WebNode shouldn't really exist in the Blink Onion Soup world. In this particular case, the knowledge of the focused element was already being plumbed in and out of Blink. This patch removes the weaving and the API that caused it.

There should be no behavior change, but one of the existing behaviors wasn't covered by tests, so I added it.

BUG=606086
TEST= WebPluginContainerTest.CopyFromContextMenu

Committed: https://crrev.com/1b1ec427a7fc6c12c6323930b93eee37ee71dc42
Cr-Commit-Position: refs/heads/master@{#393923}

[dglazkov, 2016-05-13 18:15:55]

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

[commit-bot: I haz the power, 2016-05-13 18:16:32]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977633003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977633003/1

[commit-bot: I haz the power, 2016-05-13 18:34:43]

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

[commit-bot: I haz the power, 2016-05-13 18:34:44]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dglazkov, 2016-05-13 20:16:41]

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

[dglazkov, 2016-05-13 20:16:43]

Compile.

[commit-bot: I haz the power, 2016-05-13 20:17:14]

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

[commit-bot: I haz the power, 2016-05-13 20:49:48]

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

[commit-bot: I haz the power, 2016-05-13 20:49:49]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[dglazkov, 2016-05-13 22:46:29]

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

[dglazkov, 2016-05-13 22:46:30]

Run tests.

[commit-bot: I haz the power, 2016-05-13 22:47:14]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977633003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977633003/40001

[dglazkov, 2016-05-14 00:05:47]

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

[commit-bot: I haz the power, 2016-05-14 00:06:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977633003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977633003/60001

[commit-bot: I haz the power, 2016-05-14 01:52:33]

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

[commit-bot: I haz the power, 2016-05-14 01:52:34]

Dry run: This issue passed the CQ dry run.

[dglazkov, 2016-05-14 03:46:15]

Description was changed from

==========
Remove RenderFrameImpl::GetFocusedElement.

BUG=606086
==========

to

==========
Remove RenderFrameImpl::GetFocusedElement.

The concept of WebNode shouldn't really exist in the Blink world. In this
particular case, the knowledge of the focused element was already being plumbed
in and out of Blink. This patch removes the weaving and the API that caused it.

There should be no behavior change, but one of the existing behaviors wasn't
covered by tests, so I added it.

BUG=606086
TEST= WebPluginContainerTest.CopyFromContextMenu
==========

[dglazkov, 2016-05-14 03:48:14]

dglazkov@chromium.org changed reviewers:
+ esprehn@chromium.org, estade@chromium.org, isherman@chromium.org,
jam@chromium.org

[dglazkov, 2016-05-14 03:49:25]

PTAL. The original code was added in https://codereview.chromium.org/7280001,
and was then adorned with https://codereview.chromium.org/15071004. This patch
just beats on the sheetmetal some more, straightening out the wrinkles.

[dglazkov, 2016-05-14 03:50:00]

Description was changed from

==========
Remove RenderFrameImpl::GetFocusedElement.

The concept of WebNode shouldn't really exist in the Blink world. In this
particular case, the knowledge of the focused element was already being plumbed
in and out of Blink. This patch removes the weaving and the API that caused it.

There should be no behavior change, but one of the existing behaviors wasn't
covered by tests, so I added it.

BUG=606086
TEST= WebPluginContainerTest.CopyFromContextMenu
==========

to

==========
Remove RenderFrameImpl::GetFocusedElement.

The concept of WebNode shouldn't really exist in the Blink Onion Soup world. In
this particular case, the knowledge of the focused element was already being
plumbed in and out of Blink. This patch removes the weaving and the API that
caused it.

There should be no behavior change, but one of the existing behaviors wasn't
covered by tests, so I added it.

BUG=606086
TEST= WebPluginContainerTest.CopyFromContextMenu
==========

[esprehn, 2016-05-14 04:14:51]

lgtm

[jam, 2016-05-16 15:52:16]

lgtm

[dglazkov, 2016-05-16 15:52:57]

The CQ bit was checked by dglazkov@chromium.org

[commit-bot: I haz the power, 2016-05-16 15:53:16]

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

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

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-16 21:16:01]

Description was changed from

==========
Remove RenderFrameImpl::GetFocusedElement.

The concept of WebNode shouldn't really exist in the Blink Onion Soup world. In
this particular case, the knowledge of the focused element was already being
plumbed in and out of Blink. This patch removes the weaving and the API that
caused it.

There should be no behavior change, but one of the existing behaviors wasn't
covered by tests, so I added it.

BUG=606086
TEST= WebPluginContainerTest.CopyFromContextMenu
==========

to

==========
Remove RenderFrameImpl::GetFocusedElement.

The concept of WebNode shouldn't really exist in the Blink Onion Soup world. In
this particular case, the knowledge of the focused element was already being
plumbed in and out of Blink. This patch removes the weaving and the API that
caused it.

There should be no behavior change, but one of the existing behaviors wasn't
covered by tests, so I added it.

BUG=606086
TEST= WebPluginContainerTest.CopyFromContextMenu

Committed: https://crrev.com/1b1ec427a7fc6c12c6323930b93eee37ee71dc42
Cr-Commit-Position: refs/heads/master@{#393923}
==========

[commit-bot: I haz the power, 2016-05-16 21:16:02]

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