Add DialogShowParams to <dialog>'s show() and showModal()

Add DialogShowParams to <dialog>'s show() and showModal()

Currently, this has only 1 option: skipFocusing = true/false

You invoke this via:

dialogElement.showModal({skipFocusing: true});

and set your own focus.  Omitting the DialogShowParams does nothing.

Rationale: https://github.com/whatwg/html/issues/1929
Spec addition: https://github.com/whatwg/html/pull/2197

R=tkent@chromium.org
BUG=656924

[Dan Beam, 2016-12-20 01:37:37]

Patchset #2 (id:20001) has been deleted

[Dan Beam, 2016-12-20 19:55:49]

Description was changed from

==========
Parameterize <dialog>'s show() and showModal() in order to skip initial focus
logic

R=
BUG=
==========

to

==========
Parameterize <dialog>'s show() and showModal() in order to skip initial focus
logic

Rationale: https://github.com/whatwg/html/issues/1929
Spec addition: https://github.com/whatwg/html/pull/2197

R=tkent@chromium.org
BUG=656924
==========

[Dan Beam, 2016-12-20 19:55:49]

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

[Dan Beam, 2016-12-20 19:57:22]

Description was changed from

==========
Parameterize <dialog>'s show() and showModal() in order to skip initial focus
logic

Rationale: https://github.com/whatwg/html/issues/1929
Spec addition: https://github.com/whatwg/html/pull/2197

R=tkent@chromium.org
BUG=656924
==========

to

==========
Add DialogShowParams to <dialog>'s show() and showModal()

Currently, this has only 1 option: skipFocusing = true/false

You invoke this via:

dialogElement.showModal({skipFocusing: true});

and set your own focus.  Omitting the DialogShowParams does nothing.

Rationale: https://github.com/whatwg/html/issues/1929
Spec addition: https://github.com/whatwg/html/pull/2197

R=tkent@chromium.org
BUG=656924
==========

[Dan Beam, 2016-12-20 19:58:11]

third time's a charm

[Dan Beam, 2016-12-20 19:58:19]

Patchset #1 (id:1) has been deleted

[Dan Beam, 2016-12-20 22:25:40]

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

[commit-bot: I haz the power, 2016-12-20 22:26:21]

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

[tkent, 2016-12-20 23:18:38]

Please follow a process: http://www.chromium.org/blink/launching-features
This needs a chromestatus.com entry and intent-to-implement-and-ship.

[commit-bot: I haz the power, 2016-12-21 00:47:44]

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

[commit-bot: I haz the power, 2016-12-21 00:47:44]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-12-29 22:39:45]

On 2016/12/20 23:18:38, slow_or_unavailable wrote:
> Please follow a process: http://www.chromium.org/blink/launching-features
> This needs a http://chromestatus.com entry and intent-to-implement-and-ship.

https://www.chromestatus.com/features/5121613552943104
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/Tanq8NVTar0

[Dan Beam, 2016-12-29 22:39:56]

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

[commit-bot: I haz the power, 2016-12-29 22:40: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-12-30 00:32:11]

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

[commit-bot: I haz the power, 2016-12-30 00:32:12]

Dry run: This issue passed the CQ dry run.