New issue
Advanced search Search tips

Issue 868666 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Aug 28
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-08-17
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

NEEDS_FEEDBACK endless loop inside "int TextIteratorAlgorithm<Strategy>::RangeLength"

Reported by tamir.za...@gmail.com, Jul 28

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.75 Safari/537.36

Steps to reproduce the problem:
I can reproduce ii without a problem, but unfortunately i couldn't isolate a simple example. 

What is the expected behavior?

What went wrong?
chrome_child!ChromeMainDelegate::CreateContentBrowserClient
chrome_child!blink::TextIteratorTextNodeHandler::HandleTextBox + 0x7f
chrome_child!blink::TextIteratorTextNodeHandler::HandleRemainingTextRuns + 0x27
chrome_child!blink::TextIteratorAlgorithm<blink::EditingAlgorithm<blink::NodeTraversal> >::Advance + 0x99
chrome_child!blink::TextIteratorAlgorithm<blink::EditingAlgorithm<blink::NodeTraversal> >::RangeLength + 0x79
<--!!ENDLESS LOOP HERE!!->
chrome_child!blink::PlainTextRange::Create + 0xdf
chrome_child!blink::InputMethodController::GetSelectionOffsets + 0x81
chrome_child!blink::WebInputMethodControllerImpl::GetSelectionOffsets + 0x4e
chrome_child!content::RenderFrameImpl::SyncSelectionIfRequired + 0x16c
chrome_child!blink::Editor::RespondToChangedSelection + 0x64
chrome_child!blink::FrameSelection::SetSelectionDeprecated + 0x124
chrome_child!blink::FrameSelection::SetSelection + 0x11
chrome_child!blink::SelectionController::SetNonDirectionalSelectionIfNeeded + 0x55d
chrome_child!blink::SelectionController::UpdateSelectionForMouseDownDispatchingSelectStart + 0x12e
chrome_child!blink::SelectionController::HandleSingleClick + 0x515
chrome_child!blink::SelectionController::HandleMousePressEvent + 0x120
chrome_child!blink::MouseEventManager::HandleMousePressEvent + 0x479
chrome_child!blink::EventHandler::HandleMousePressEvent + 0x85c
chrome_child!blink::PageWidgetEventHandler::HandleMouseDown + 0x41
chrome_child!blink::WebViewImpl::HandleMouseDown + 0x277
chrome_child!blink::PageWidgetDelegate::HandleInputEvent + 0x439
chrome_child!blink::WebViewImpl::HandleInputEvent + 0x35e
chrome_child!content::RenderWidgetInputHandler::HandleInputEvent + 0x2ac
chrome_child!content::RenderWidget::HandleInputEvent + 0x4c
chrome_child!content::RenderViewImpl::HandleInputEvent + 0x7d
chrome_child!content::MainThreadEventQueue::HandleEventOnMainThread + 0x52
chrome_child!content::QueuedWebInputEvent::Dispatch + 0xbf
chrome_child!content::MainThreadEventQueue::DispatchEvents + 0x114
chrome_child!base::debug::TaskAnnotator::RunTask + 0x135
chrome_child!base::sequence_manager::internal::ThreadControllerImpl::DoWork + 0x173
chrome_child!base::debug::TaskAnnotator::RunTask + 0x135
chrome_child!base::MessageLoop::RunTask + 0x23c
chrome_child!base::MessageLoop::DoWork + 0x198
chrome_child!base::MessagePumpDefault::Run + 0x99
chrome_child!base::RunLoop::Run + 0x31
chrome_child!content::RendererMain + 0x519
chrome_child!content::ContentMainRunnerImpl::Run + 0x1e5
chrome_child!service_manager::Main + 0x6ed
chrome_child!content::ContentMain + 0x41
chrome_child!ChromeMain + 0x120
chrome!MainDllLoader::Launch + 0x26c
chrome!wWinMain + 0x699
chrome!__scrt_common_main_seh + 0x106
KERNEL32!BaseThreadInitThunk + 0x14
ntdll!RtlUserThreadStart + 0x21

Did this work before? No 

Chrome version: 68.0.3440.75  Channel: stable
OS Version: 10.0
Flash Version: 

This happens on a site that available online, but requires username + password.
 
please excuse me, I'm hardly an "expert" in such a "complex" debugging, and this is the first bug I report...

Anyway, I was eventually able debug it using vs 2017, and it was clear that the endless loop was inside the method
"int TextIteratorAlgorithm<Strategy>::RangeLength"
inside the file:
"third_party\blink\renderer\core\editing\iterators".

This happens in more then 3 different computers in chrome version "68.0.3440.75".

But it does not! happen in canary:
"Version 70.0.3505.0 (Official Build) canary (64-bit)"

Please, help me!?! (:
10x
...inside the file:
"...iterators\text_iterator.cc"...
Components: -Blink Blink>Editing
Cc: susan.boorgula@chromium.org
Labels: Triaged-ET Needs-Triage-M68 Needs-Feedback
tamir.zaslavsky@ Thanks for the issue.

Request you to provide a sample File/URL where this issue can be reproduced, which will help in further triaging.

Thanks..
HI susan.boorgula@chromium.org (:

Unfortunately, I'm not allowed to so since it requires me to "expose" username + password to one of my client.

Although I have rich experience in debugging, as I wrote above, until now I never needed to debug such a "complex" code.

For example, the some inline methods really got me confused because in many case my IDE (vs2017) didn't colored the "actual executing line" and also prevented me to be able to add break points so i can pinpoint the root cause.

Of course after some reading I'm now understand more, but the point here is solving the bug and less educating me.. (:

So, I'm more than happy to "remote debug it for you" If you are willing to instruct me.

Maybe be there is a way to:

debug some iterations of the loop and also "record" all variables, and then sent it?

Or maybe more primitively, just record my screen while doing a debug?
Components: -Blink>Editing Blink>Editing>Serialization
NextAction: 2018-08-17
Summary: NEEDS_FEEDBACK endless loop inside "int TextIteratorAlgorithm<Strategy>::RangeLength" (was: endless loop inside "int TextIteratorAlgorithm<Strategy>::RangeLength")
tamir.zaslavsky@, could you provide HTML which causes this infinite loop?
We don't need to use real site. We just need to have HTML+CSS.

1. Hit Ctrl+S to open "Save As" dialog
2. Choose "Webpage complete" to include both HTML and CSS, images may not needed.
3. ZIP them
4. Attach here.

Thanks in advance.
The NextAction date has arrived: 2018-08-17
Status: WontFix (was: Unconfirmed)
Mark WontFix since no feedback almost a month.
Once you get reproduce case, please reopen this issue.
Thanks!

Sign in to add a comment