stylo: Crashes while loading alexa real site snapshots

RESOLVED WORKSFORME

Status

()

Core
CSS Parsing and Computation
P1
normal
RESOLVED WORKSFORME
5 months ago
2 months ago

People

(Reporter: shinglyu, Unassigned)

Tracking

(Blocks: 1 bug, {meta})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 months ago
While testing the websites from bug 1331552, I found a few crashes:

aarp.org.html
airberlin.com.html
bankrate.com.html
bbb.org.html
bbc.com-footbal.html
cafepress.com.html
cbssports.com.html
cdc.gov.html
census.gov.html
cj.com.html
deadspin.com.html
discovery.com.html
download.cnet.com.html
edmunds.com.html
emedtv.com.html
epinions.com-digital-cameras.html
(Reporter)

Comment 1

5 months ago
Chris, do you think we can put the zip of saved pages here? Or maybe we send it through internal mailing list to avoid copyright issue?
Flags: needinfo?(cpeterson)
I don't think we can attach the zip of saved pages in Bugzilla because of the copyright issues and the huge file size. I'll upload the zip to Mozilla's Google Drive.
Flags: needinfo?(cpeterson)
(Reporter)

Comment 3

5 months ago
More crashes I missed:

amextravel.com.html
bbc.com-news.html
cafemom.com.html
chicagotribune.com.html
Keywords: meta
Priority: -- → P2
Do we still have crashes on alexa? If so please file them as stylo: bugs and mark them p1.
Flags: needinfo?(shing.lyu)
(Reporter)

Updated

2 months ago
Flags: needinfo?(shing.lyu)
Priority: P2 → P1
Summary: [Stylo] Crashes while loading real site snapshots → stylo: Crashes while loading alexa real site snapshots
(Reporter)

Comment 5

2 months ago
Created attachment 8874358 [details]
crashes list

This is the latest crashes from linux64 stylo opt build.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=09c0a641034905c64e2675fee0cfbf49b845eff8
(Reporter)

Comment 6

2 months ago
In debug build, ebay.com-auction.html is passing but apa.org.html is crashing.
Can you file the crashes as bugs with crash stacks? No need to file for each site, just for each failure mode.
Flags: needinfo?(shing.lyu)
(Reporter)

Comment 8

2 months ago
After looking into each one, I found they are all false alarms. The reasons:

* Snapshots contains external links, so reftest blocks them for security reason
4shared.com.html
chicagotribune.com.html
couchsurfing.com.html
drbatras.com.html
edmunds.com.html
freelancer.com-home.html
freelancer.com-jobs.html
gamehouse.com.html

* JS error makes reftest believe it never finished loading => timeout
bankrate.com.html
barnesandnoble.com.html
bookish.com.html
cafemom.com.html
cbssports.com.html
census.gov.html
fool.com.html
forum.xda-developers.com.html
foxnews.com-entertainment.html
foxnews.com-home.html
foxnews.com-politics.html
my.xfinity.com.html

* JS alert() blocks reftest loading
apa.org.html

* The text appeared in other error log, but itself is not crashing
gap.com.html


I think it's safe to mark this as resloved worksforme for now. I'll keep monitoring their status.
Flags: needinfo?(shing.lyu)
(Reporter)

Updated

2 months ago
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → WORKSFORME
\o/ Great news!
You need to log in before you can comment on or make changes to this bug.