[Editing] Make InsertListCommand check endingSelection() editability.

[Editing] Make InsertListCommand check endingSelection() editability.

doApply assumes |endingSelection()| has rootEditableElement as L114 checks.
However, reseting |endingSelection()| through |endOfSelection| at L171 can 
invalidate the assumption.
This CL let it return early if it does.
We should fix the reseting |endingSelection| algorithm in long term but let me fix 
the stable blocker crash.

BUG=515490
TEST=LayoutTests/editing/execCommand/insert-ordered-list-crash2.html

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

[yoichio, 2015-08-18 01:23:52]

Patchset #1 (id:1) has been deleted

[yoichio, 2015-08-18 01:33:46]

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

[yosin_UTC9, 2015-08-18 02:05:45]

lgtm

Please update description to fin in 80 chars line.

Replace
This CL let => This CL lets

Add 
TEST=LayoutTests/editing/execCommand/insert-ordered-list-crash2.html

It is better to describe what this patch does rather than
just say "fix crash", e.g.

Make InsertListCommand not to modify uneditable part

Or another, what this patch changes to do.

https://codereview.chromium.org/1298633002/diff/20001/LayoutTests/editing/exe...
File LayoutTests/editing/execCommand/insert-ordered-list-crash2.html (right):

https://codereview.chromium.org/1298633002/diff/20001/LayoutTests/editing/exe...
LayoutTests/editing/execCommand/insert-ordered-list-crash2.html:13: <div
>DDD</div>
nit: Do we need to have a space in "<div >"?

[yoichio, 2015-08-18 05:46:25]

Update description.
Thanks

https://codereview.chromium.org/1298633002/diff/20001/LayoutTests/editing/exe...
File LayoutTests/editing/execCommand/insert-ordered-list-crash2.html (right):

https://codereview.chromium.org/1298633002/diff/20001/LayoutTests/editing/exe...
LayoutTests/editing/execCommand/insert-ordered-list-crash2.html:13: <div
>DDD</div>
On 2015/08/18 02:05:45, Yosi_UTC9 wrote:
> nit: Do we need to have a space in "<div >"?

Done.

[yoichio, 2015-08-18 05:46:30]

The CQ bit was checked by yoichio@chromium.org

[yoichio, 2015-08-18 05:46:31]

The patchset sent to the CQ was uploaded after l-g-t-m from yosin@chromium.org
Link to the patchset: https://codereview.chromium.org/1298633002/#ps40001
(title: "nit pick")

[commit-bot: I haz the power, 2015-08-18 05:46:38]

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

[commit-bot: I haz the power, 2015-08-18 07:13:45]

Committed patchset #2 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200709