compositionstart event's data should contain text to replace.

compositionstart event's data should contain text to replace.
spec:
http://www.w3.org/TR/DOM-Level-3-Events/#event-type-compositionstart

As far as I tested, IE10 on Windows7 and Firefox22 on Linux
conforms the spec's behavior about event.data of compositionstart event.

BUG=247878
TEST=layoutest

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153319

[kochi, 2013-06-27 10:10:33]

tkent-san,

Could you review this?

[kochi, 2013-06-27 20:53:07]

On 2013/06/27 10:10:33, Takayoshi Kochi wrote:
> tkent-san,
> 
> Could you review this?

I'll add cases for when some text is selected when compositionstart event fires.
Please hold the review.

[kochi, 2013-06-28 06:55:07]

On 2013/06/27 20:53:07, Takayoshi Kochi wrote:
> On 2013/06/27 10:10:33, Takayoshi Kochi wrote:
> > tkent-san,
> > 
> > Could you review this?
> 
> I'll add cases for when some text is selected when compositionstart event
fires.
> Please hold the review.

Done.
Could you review this?

[tkent, 2013-06-30 23:32:32]

Do other browser conform to the specification?

[kochi, 2013-07-01 03:48:36]

On 2013/06/30 23:32:32, tkent wrote:
> Do other browser conform to the specification?

IE10 and the latest firefox conform the spec, as far as I know.

[tkent, 2013-07-01 03:51:07]

lgtm

> IE10 and the latest firefox conform the spec, as far as I know.

Please add this information to the CL description.

[kochi, 2013-07-01 03:59:29]

On 2013/07/01 03:51:07, tkent wrote:
> lgtm
> 
> > IE10 and the latest firefox conform the spec, as far as I know.
> 
> Please add this information to the CL description.

Thanks for the review!
Updated the description.

[commit-bot: I haz the power, 2013-07-01 03:59:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kochi@chromium.org/18041002/10001

[commit-bot: I haz the power, 2013-07-01 05:34:52]

Change committed as 153319

[rniwa-cr, 2013-07-15 21:46:52]

https://chromiumcodereview.appspot.com/18041002/diff/10001/Source/core/editin...
File Source/core/editing/Editor.cpp (right):

https://chromiumcodereview.appspot.com/18041002/diff/10001/Source/core/editin...
Source/core/editing/Editor.cpp:1256:
target->dispatchEvent(CompositionEvent::create(eventNames().compositionstartEvent,
m_frame->document()->domWindow(), selectionText));
You could have simply called selectedText() here as I've done so in
https://bugs.webkit.org/show_bug.cgi?id=118684.

[kochi, 2013-07-16 09:50:41]

https://chromiumcodereview.appspot.com/18041002/diff/10001/Source/core/editin...
File Source/core/editing/Editor.cpp (right):

https://chromiumcodereview.appspot.com/18041002/diff/10001/Source/core/editin...
Source/core/editing/Editor.cpp:1256:
target->dispatchEvent(CompositionEvent::create(eventNames().compositionstartEvent,
m_frame->document()->domWindow(), selectionText));
On 2013/07/15 21:46:52, rniwa-cr wrote:
> You could have simply called selectedText() here as I've done so in
> https://bugs.webkit.org/show_bug.cgi?id=118684.

Done.

[kochi, 2013-07-16 09:50:56]

On 2013/07/16 09:50:41, Takayoshi Kochi wrote:
>
https://chromiumcodereview.appspot.com/18041002/diff/10001/Source/core/editin...
> File Source/core/editing/Editor.cpp (right):
> 
>
https://chromiumcodereview.appspot.com/18041002/diff/10001/Source/core/editin...
> Source/core/editing/Editor.cpp:1256:
>
target->dispatchEvent(CompositionEvent::create(eventNames().compositionstartEvent,
> m_frame->document()->domWindow(), selectionText));
> On 2013/07/15 21:46:52, rniwa-cr wrote:
> > You could have simply called selectedText() here as I've done so in
> > https://bugs.webkit.org/show_bug.cgi?id=118684.
> 
> Done.

The CL is https://codereview.chromium.org/19357002/