1341997 - stylo: Crashes while loading alexa real site snapshots

Status: RESOLVED WORKSFORME

(Core :: CSS Parsing and Computation, defect, P1)

Description

[Shing Lyu [:shinglyu]]

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

Comment 1

[Shing Lyu [:shinglyu]]

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?

Comment 2

[Chris Peterson [: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.

Comment 3

[Shing Lyu [:shinglyu]]

More crashes I missed:

amextravel.com.html
bbc.com-news.html
cafemom.com.html
chicagotribune.com.html

Comment 4

[Bobby Holley (:bholley)]

Do we still have crashes on alexa? If so please file them as stylo: bugs and mark them p1.

Comment 5

[Shing Lyu [:shinglyu]]

Attachment: crashes list

This is the latest crashes from linux64 stylo opt build.

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

Comment 6

[Shing Lyu [:shinglyu]]

In debug build, ebay.com-auction.html is passing but apa.org.html is crashing.

Comment 7

[Bobby Holley (:bholley)]

Can you file the crashes as bugs with crash stacks? No need to file for each site, just for each failure mode.

Comment 8

[Shing Lyu [:shinglyu]]

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.

Comment 9

[Bobby Holley (:bholley)]

\o/ Great news!