Commit 113aa061 authored by Shimi Zhang's avatar Shimi Zhang Committed by Commit Bot

Add log to collect more data for a crash

The current crash report tells mostly browser side info.

Bug: 850954
Change-Id: I2d479e6b7664d38939d498b34a86543521e6fc42
Reviewed-on: https://chromium-review.googlesource.com/1222251Reviewed-by: default avatarDave Tapuska <dtapuska@chromium.org>
Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#591136}
parent 25b9a8e6
......@@ -382,6 +382,15 @@ bool ContextMenuController::ShowContextMenu(LocalFrame* frame,
WebRange range =
selected_frame->GetInputMethodController().GetSelectionOffsets();
data.selection_start_offset = range.StartOffset();
// TODO(crbug.com/850954): Remove redundant log after we identified the
// issue.
DCHECK_GE(data.selection_start_offset, 0)
<< "Log issue against https://crbug.com/850954\n"
<< "data.selection_start_offset: " << data.selection_start_offset
<< "\nrange: [" << range.StartOffset() << ", " << range.EndOffset()
<< "]\nVisibleSelection: "
<< selected_frame->Selection()
.ComputeVisibleSelectionInDOMTreeDeprecated();
}
if (result.IsContentEditable()) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment