Make FrameSelection::SelectAll bail if 'selectstart' detaches frame

Make FrameSelection::SelectAll bail if 'selectstart' detaches frame

BUG=715387
TEST=LayoutTests/editing/selection/selectstart_detaches_frame.html

Review-Url: https://codereview.chromium.org/2848993002
Cr-Commit-Position: refs/heads/master@{#468292}
Committed: https://chromium.googlesource.com/chromium/src/+/73fe6ba4ceb2db91a43ee2305f69ffde08001f62

[Xiaocheng, 2017-04-29 00:42:56]

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

[commit-bot: I haz the power, 2017-04-29 00:43:32]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2017-04-29 00:43:38]

Description was changed from

==========
Make FrameSelection::SelectAll bail if selectstart destroys frame

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

to

==========
Make FrameSelection::SelectAll bail if 'selectstart' detaches frame

BUG=715387
TEST=LayoutTests/editing/selection/selectstart_detaches_frame.html
==========

[Xiaocheng, 2017-04-29 01:01:47]

xiaochengh@chromium.org changed reviewers:
+ kojii@chromium.org

[Xiaocheng, 2017-04-29 01:01:47]

PTAL, as both yosin@ and tkent@ are OOO...

[commit-bot: I haz the power, 2017-04-29 02:32:07]

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

[commit-bot: I haz the power, 2017-04-29 02:32:07]

Dry run: This issue passed the CQ dry run.

[kojii, 2017-05-01 01:58:18]

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

[kojii, 2017-05-01 01:58:18]

tkent@, PTAL, and here.

[tkent, 2017-05-01 02:25:02]

lgtm

[Xiaocheng, 2017-05-01 02:30:11]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2017-05-01 02:30:28]

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, 2017-05-01 03:45:55]

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

[commit-bot: I haz the power, 2017-05-01 03:45:56]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[xiaochengh, 2017-05-01 04:38:57]

The CQ bit was checked by xiaochengh@google.com

[commit-bot: I haz the power, 2017-05-01 04:39:06]

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, 2017-05-01 05:42:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493613537829080, "parent_rev":
"d62951aca80fa961d7feb4c05fe62b0893f59bfd", "commit_rev":
"73fe6ba4ceb2db91a43ee2305f69ffde08001f62"}

[commit-bot: I haz the power, 2017-05-01 05:43:10]

Description was changed from

==========
Make FrameSelection::SelectAll bail if 'selectstart' detaches frame

BUG=715387
TEST=LayoutTests/editing/selection/selectstart_detaches_frame.html
==========

to

==========
Make FrameSelection::SelectAll bail if 'selectstart' detaches frame

BUG=715387
TEST=LayoutTests/editing/selection/selectstart_detaches_frame.html

Review-Url: https://codereview.chromium.org/2848993002
Cr-Commit-Position: refs/heads/master@{#468292}
Committed:
https://chromium.googlesource.com/chromium/src/+/73fe6ba4ceb2db91a43ee2305f69...
==========

[commit-bot: I haz the power, 2017-05-01 05:43:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/73fe6ba4ceb2db91a43ee2305f69...