Closed Bug 1244798 Opened 7 years ago Closed 7 years ago

Crash when using Find in Page


(Core :: DOM: Core & HTML, defect)

Not set



Tracking Status
e10s ? ---
firefox47 + fixed


(Reporter: armenzg, Assigned: khuey)




(Keywords: crash, crashreportid)

Crash Data


(1 file)

Since Monday 25th, I'm experiencing all my tabs crashing at once multiple times a day.

I'm running on Linux, using Nightly, only JSONView as my active add-on.

This would generally happen when pastebin or treeherder would be loading.
I have noticed that if I try to search in the page it will consistenly crash all my tabs.

Raising to major.
Severity: normal → major
Duplicate of this bug: 1245014
Component: Find Toolbar → General
Product: Toolkit → Core
Version: unspecified → Trunk
Crash Signature: [@ nsTypeAheadFind::FindItNow ]
Has STR: --- → yes
Summary: All my tabs have been crashing recently → Crash when using Find in Page
Trying to find a regression range.
Severity: major → critical
Flags: needinfo?(mconley)
I can't seem to reproduce this on my Ubuntu VM. jdm, if you have a sec, if you're able to reproduce this reliably, do you think you can mozregression us a range? I figure this has got to be pretty recent.
Flags: needinfo?(mconley) → needinfo?(josh)
(In reply to Milan Sreckovic [:milan] from comment #6)
> I hit this twice today, but now can't reproduce on demand:
> 0e7d12160203
> 39c542160203

These on OS X.
Reproduced on windows
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:47.0) Gecko/20100101 Firefox/47.0 ID:20160203030249
OS: Unspecified → All
Crash Signature: [@ nsTypeAheadFind::FindItNow ] → [@ nsTypeAheadFind::FindItNow ] [@ nsPIDOMWindow<T>::GetDocShell]

FWIW Some steps that crash reliably here.

Press <Ctrl>+<F>
Type "QuitProgressUI"
Click down arrow until the search cycles to the top

Tab crashes

Expected result:
Top most search result is highlighted

Thanks for the STR! Those work for me.

My instinct puts my suspicions on bug 1241764.

Using mozregression - stand by...
Flags: needinfo?(josh)
Here's the regression range:

Hey khuey - I suspect your work in bug 1241764 caused this... do the stacks make that plausible?

[Tracking Requested - why for this release]:

Pretty easy-to-hit crash regression.
Blocks: 1241764
Flags: needinfo?(khuey)
Attached patch PatchSplinter Review
Assignee: nobody → khuey
Flags: needinfo?(khuey)
Attachment #8715483 - Flags: review?(bugs)
Component: General → DOM
Attachment #8715483 - Flags: review?(bugs) → review+
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Looking forward to the next nightly! Thanks for the fix!

Is there a test that could have caught this?
If I understand the STR from comment 10 correctly, then an automated test that does a Find Bar search that hops over iframes would probably have caused the crash to appear in automation.

As it stands, I don't believe we have such a test - though I'd be happy to mentor somebody in writing one.
I would be happy to learn. It might take a bit of time and I'm going in pat leave in March but if possible I would love to learn.
I filed bug 1247948 separetely.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.