[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F[1] and GitHub discussion[2])
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion[3])
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

[1] Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOwuQ/edit#heading=h.l9vlzb1oc68r

[2] GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

[3] Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEvent/blink-dev/RrnitB0OElc/rirueVekCwAJ

BUG=639139
Committed: https://crrev.com/952c4101425ed85bf53ded777e5a882895646980
Cr-Commit-Position: refs/heads/master@{#414659}

[chongz, 2016-08-18 20:40:19]

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

[commit-bot: I haz the power, 2016-08-18 20:40:43]

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-08-18 22:18:04]

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

[commit-bot: I haz the power, 2016-08-18 22:18:05]

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

[chongz, 2016-08-18 23:41:44]

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

[commit-bot: I haz the power, 2016-08-18 23:42:34]

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

[chongz, 2016-08-18 23:57:34]

Description was changed from

==========
Added |dataTransfer| field to beforeinput cut/paste

BUG=
==========

to

==========
[InputEvent] Support cut/paste with |dataTransfer| field and correct 
event order

* Event order: (see July F2F)
  1. 'beforeinput' cut
  2. Clipboard API cut
  3. Dom update & clipboard update
  4. 'input' cut

* Canceling 'beforeinput' will prevent all following actions.

* |dataTransfer| field: (see Editing spec discussion)
  1. NULL for cut (JS could always get from selection)
  2. Readonly fragment for paste

Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

BUG=639139
==========

[chongz, 2016-08-18 23:57:51]

Description was changed from

==========
[InputEvent] Support cut/paste with |dataTransfer| field and correct 
event order

* Event order: (see July F2F)
  1. 'beforeinput' cut
  2. Clipboard API cut
  3. Dom update & clipboard update
  4. 'input' cut

* Canceling 'beforeinput' will prevent all following actions.

* |dataTransfer| field: (see Editing spec discussion)
  1. NULL for cut (JS could always get from selection)
  2. Readonly fragment for paste

Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

BUG=639139
==========

to

==========
[InputEvent] Support cut/paste with |dataTransfer| field and correct event order

* Event order: (see July F2F)
  1. 'beforeinput' cut
  2. Clipboard API cut
  3. Dom update & clipboard update
  4. 'input' cut

* Canceling 'beforeinput' will prevent all following actions.

* |dataTransfer| field: (see Editing spec discussion)
  1. NULL for cut (JS could always get from selection)
  2. Readonly fragment for paste

Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

BUG=639139
==========

[chongz, 2016-08-18 23:58:33]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-08-19 04:28:15]

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

[commit-bot: I haz the power, 2016-08-19 04:28:16]

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

[chongz, 2016-08-23 00:34:38]

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

[commit-bot: I haz the power, 2016-08-23 00:35:14]

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-08-23 02:08:29]

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

[commit-bot: I haz the power, 2016-08-23 02:08:30]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[chongz, 2016-08-23 03:53:46]

Patchset #1 (id:20001) has been deleted

[chongz, 2016-08-23 03:59:47]

Description was changed from

==========
[InputEvent] Support cut/paste with |dataTransfer| field and correct event order

* Event order: (see July F2F)
  1. 'beforeinput' cut
  2. Clipboard API cut
  3. Dom update & clipboard update
  4. 'input' cut

* Canceling 'beforeinput' will prevent all following actions.

* |dataTransfer| field: (see Editing spec discussion)
  1. NULL for cut (JS could always get from selection)
  2. Readonly fragment for paste

Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

BUG=639139
==========

to

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F and GitHub discussion)
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion)
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139
==========

[chongz, 2016-08-23 04:01:14]

Description was changed from

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F and GitHub discussion)
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion)
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139
==========

to

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F[1] and GitHub discussion[2])
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion[3])
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

[1] Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

[2] GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

[3] Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139
==========

[chongz, 2016-08-23 04:04:29]

chongz@chromium.org changed reviewers:
+ yosin@chromium.org

[chongz, 2016-08-23 04:04:30]

yosin@ PTAL at this 'beforeinput' CL for 'cut' and 'paste', thanks!
(Will take a look at 'linux_android_rel_ng' if it still fails.)

[yosin_UTC9, 2016-08-23 04:23:54]

https://codereview.chromium.org/2258663003/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/events/inputevents/inputevent-cut-paste.html
(right):

https://codereview.chromium.org/2258663003/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/inputevents/inputevent-cut-paste.html:1:
<!DOCTYPE html>
Could you add a test case which destroy a frame containing an element causing
"beforeinput"?

https://codereview.chromium.org/2258663003/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/Editor.cpp (right):

https://codereview.chromium.org/2258663003/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/Editor.cpp:912: if (!m_frame)
|Editor::m_frame| is never |nullptr|.
You can check |m_frame| is whether |LocalFrame| is available or not by:

Document* document = m_frame->document();
if (disaptchBeforeInputDataTransfer(...))
   return;
if (m_frame->document() != document)
  return;

[chongz, 2016-08-23 16:44:16]

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

[commit-bot: I haz the power, 2016-08-23 16:44:39]

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-08-23 16:47:28]

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

[commit-bot: I haz the power, 2016-08-23 16:47:29]

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

[chongz, 2016-08-23 19:29:45]

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

[chongz, 2016-08-23 19:30:07]

Patchset #2 (id:60001) has been deleted

[commit-bot: I haz the power, 2016-08-23 19:30:30]

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-08-23 21:24:38]

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

[commit-bot: I haz the power, 2016-08-23 21:24:39]

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

[chongz, 2016-08-23 22:54:42]

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

[commit-bot: I haz the power, 2016-08-23 22:55:28]

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

[chongz, 2016-08-23 23:41:18]

Patchset #2 (id:80001) has been deleted

[chongz, 2016-08-23 23:41:31]

Patchset #2 (id:100001) has been deleted

[chongz, 2016-08-24 01:43:27]

yosin@ I've added iframe tests and updated crash check, PTAL, thanks!

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/editing/Editor.cpp (right):

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/editing/Editor.cpp:893: return;
Compare document won't work, have to do this...

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp (right):

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp:1885: if
(!m_frame->document()->frame())
Same here for other editing commands.

[yosin_UTC9, 2016-08-24 02:11:44]

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html
(right):

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:19:
childDocument.open();
nit: You could write:
childDocument.body.innerHTML = '<p id="editable" contenteditable>Foo</p>';

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:27:
if (actualBeforeInputCount == expectedBeforeInputCount && iframe.parentNode) {
nit: We don't need to have {} for one statement then-clause.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:28:
iframe.parentNode.removeChild(iframe);
nit: iframe.remove() is shorter.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:35:
if (iframe.parentNode) {
nit: We don't need to have {} for one statement then-clause.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:36:
iframe.parentNode.removeChild(iframe);
nit: iframe.remove() is shorter.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/editing/Editor.cpp (right):

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/editing/Editor.cpp:893: return;
On 2016/08/24 at 01:43:27, chongz wrote:
> Compare document won't work, have to do this...

How about

|m_frame->document()->frame() != m_frame|?

[chongz, 2016-08-24 02:38:59]

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

[commit-bot: I haz the power, 2016-08-24 02:39:39]

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

[chongz, 2016-08-24 02:42:01]

yosin@, I've updated as per your comments, PTAL again, thanks!

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html
(right):

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:19:
childDocument.open();
On 2016/08/24 02:11:44, Yosi_UTC9 wrote:
> nit: You could write:
> childDocument.body.innerHTML = '<p id="editable" contenteditable>Foo</p>';

Done.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:27:
if (actualBeforeInputCount == expectedBeforeInputCount && iframe.parentNode) {
On 2016/08/24 02:11:44, Yosi_UTC9 wrote:
> nit: We don't need to have {} for one statement then-clause.

Done.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:28:
iframe.parentNode.removeChild(iframe);
On 2016/08/24 02:11:44, Yosi_UTC9 wrote:
> nit: iframe.remove() is shorter.

Done.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:35:
if (iframe.parentNode) {
On 2016/08/24 02:11:44, Yosi_UTC9 wrote:
> nit: We don't need to have {} for one statement then-clause.

Done.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/inputevents/beforeinput-remove-iframe-crash.html:36:
iframe.parentNode.removeChild(iframe);
On 2016/08/24 02:11:44, Yosi_UTC9 wrote:
> nit: iframe.remove() is shorter.

Done.

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/editing/Editor.cpp (right):

https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/editing/Editor.cpp:893: return;
On 2016/08/24 02:11:44, Yosi_UTC9 wrote:
> On 2016/08/24 at 01:43:27, chongz wrote:
> > Compare document won't work, have to do this...
> 
> How about
> 
> |m_frame->document()->frame() != m_frame|?

Done.

[yosin_UTC9, 2016-08-24 03:24:55]

lgtm

[chongz, 2016-08-24 03:42:57]

chongz@chromium.org changed reviewers:
+ ojan@chromium.org

[chongz, 2016-08-24 03:42:58]

ojan@ PTAL at this 'beforeinput' CL for cut and paste, thanks!

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

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

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

Dry run: This issue passed the CQ dry run.

[chongz, 2016-08-25 23:34:28]

On 2016/08/24 03:42:58, chongz wrote:
> ojan@ PTAL at this 'beforeinput' CL for cut and paste, thanks!

ojan@ can you take a look at this CL please? Thanks!

[ojan, 2016-08-26 00:06:43]

The CQ bit was checked by ojan@chromium.org

[ojan, 2016-08-26 00:06:44]

lgtm

[commit-bot: I haz the power, 2016-08-26 00:07:59]

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-08-26 02:10:01]

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

[commit-bot: I haz the power, 2016-08-26 02:10:02]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[chongz, 2016-08-26 05:53:54]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2016-08-26 05:54:08]

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-08-26 06:33:46]

Description was changed from

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F[1] and GitHub discussion[2])
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion[3])
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

[1] Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

[2] GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

[3] Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139
==========

to

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F[1] and GitHub discussion[2])
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion[3])
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

[1] Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

[2] GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

[3] Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139
==========

[commit-bot: I haz the power, 2016-08-26 06:33:47]

Committed patchset #3 (id:140001)

[commit-bot: I haz the power, 2016-08-26 06:35:25]

Description was changed from

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F[1] and GitHub discussion[2])
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion[3])
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

[1] Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

[2] GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

[3] Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139
==========

to

==========
[InputEvent] Support |deleteByCut|&|insertFromPaste| with |dataTransfer| field

* Event order: (see July F2F[1] and GitHub discussion[2])
  1. Clipboard API cut
  2. (Clipboard update)
  3. 'beforeinput' InputType=|deleteByCut|
  4. (DOM update)
  5. 'input' InputType=|deleteByCut|

* Canceling 'beforeinput' will only prevent DOM update.

* |dataTransfer| field: (see Editing spec discussion[3])
  1. NULL for |deleteByCut| (JS could always get from selection)
  2. Readonly |dataTransfer| for |insertFromPaste|

[1] Editing TF July F2F:
https://docs.google.com/document/d/1XxIEF0So-kMF5mcJ03Yj0zsYMFRHEgXw1fV1K5FOw...

[2] GitHub discussion for event order:
https://github.com/w3c/editing/issues/144#issuecomment-240892363

[3] Editing spec discussion:
https://github.com/w3c/editing/issues/144#issuecomment-240259209

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/InputEven...

BUG=639139

Committed: https://crrev.com/952c4101425ed85bf53ded777e5a882895646980
Cr-Commit-Position: refs/heads/master@{#414659}
==========

[commit-bot: I haz the power, 2016-08-26 06:35:26]

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