Handle focusability for plugin elements which has browsing context

Handle focusability for plugin elements which has browsing context

If <object> or <embed> has HTML/SVG contents,
focus navigation within documents under these
elements should work.

This has been working fine for <iframe>s.

The FocusController code checks for FrameOwnerElement,
which is parent of <iframe>, <object>, and <embed> if
it is isKeyboardFocusable() and has contentFrame().

HTMLPluginElement::isKeyboardFocsuable() didn't handle
the cases with HTML/SVG contents, only considered plugins.

BUG=620488
Committed: https://crrev.com/a2c82afad86cf4f85f91df76f858434b15fe6b13
Cr-Commit-Position: refs/heads/master@{#404324}

[kochi, 2016-07-07 09:33:56]

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

[commit-bot: I haz the power, 2016-07-07 09:34:12]

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-07-07 10:47:51]

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

[commit-bot: I haz the power, 2016-07-07 10:47:51]

Dry run: This issue passed the CQ dry run.

[kochi, 2016-07-08 05:07:35]

Description was changed from

==========
Handle focusability for plugin elements which has browsing context

BUG=620488
==========

to

==========
Handle focusability for plugin elements which has browsing context

If <object> or <embed> has HTML/SVG contents,
focus navigation within documents under these
elements should work.

This has been working fine for <iframe>s.

The FocusController code checks for FrameOwnerElement,
which is parent of <iframe>, <object>, and <embed> if
it is isKeyboardFocusable() and has contentFrame().

HTMLPluginElement::isKeyboardFocsuable() didn't handle
the cases with HTML/SVG contents, only considered plugins.

BUG=620488
==========

[kochi, 2016-07-08 05:07:35]

kochi@chromium.org changed reviewers:
+ tkent@chromium.org

[kochi, 2016-07-08 05:15:48]

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

[kochi, 2016-07-08 05:16:13]

Could you review this?

[commit-bot: I haz the power, 2016-07-08 05:16:27]

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

[tkent, 2016-07-08 05:34:52]

lgtm

[kochi, 2016-07-08 05:39:50]

The CQ bit was unchecked by kochi@chromium.org

[kochi, 2016-07-08 05:39:50]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2016-07-08 05:40:24]

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-07-08 09:34:06]

Description was changed from

==========
Handle focusability for plugin elements which has browsing context

If <object> or <embed> has HTML/SVG contents,
focus navigation within documents under these
elements should work.

This has been working fine for <iframe>s.

The FocusController code checks for FrameOwnerElement,
which is parent of <iframe>, <object>, and <embed> if
it is isKeyboardFocusable() and has contentFrame().

HTMLPluginElement::isKeyboardFocsuable() didn't handle
the cases with HTML/SVG contents, only considered plugins.

BUG=620488
==========

to

==========
Handle focusability for plugin elements which has browsing context

If <object> or <embed> has HTML/SVG contents,
focus navigation within documents under these
elements should work.

This has been working fine for <iframe>s.

The FocusController code checks for FrameOwnerElement,
which is parent of <iframe>, <object>, and <embed> if
it is isKeyboardFocusable() and has contentFrame().

HTMLPluginElement::isKeyboardFocsuable() didn't handle
the cases with HTML/SVG contents, only considered plugins.

BUG=620488
==========

[commit-bot: I haz the power, 2016-07-08 09:34:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-08 09:34:21]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-08 09:36:15]

Description was changed from

==========
Handle focusability for plugin elements which has browsing context

If <object> or <embed> has HTML/SVG contents,
focus navigation within documents under these
elements should work.

This has been working fine for <iframe>s.

The FocusController code checks for FrameOwnerElement,
which is parent of <iframe>, <object>, and <embed> if
it is isKeyboardFocusable() and has contentFrame().

HTMLPluginElement::isKeyboardFocsuable() didn't handle
the cases with HTML/SVG contents, only considered plugins.

BUG=620488
==========

to

==========
Handle focusability for plugin elements which has browsing context

If <object> or <embed> has HTML/SVG contents,
focus navigation within documents under these
elements should work.

This has been working fine for <iframe>s.

The FocusController code checks for FrameOwnerElement,
which is parent of <iframe>, <object>, and <embed> if
it is isKeyboardFocusable() and has contentFrame().

HTMLPluginElement::isKeyboardFocsuable() didn't handle
the cases with HTML/SVG contents, only considered plugins.

BUG=620488

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

[commit-bot: I haz the power, 2016-07-08 09:36:16]

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