Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Servo shows grey screen instead of the monsterproof page #7269

Closed
suhr opened this issue Aug 18, 2015 · 22 comments
Closed

Servo shows grey screen instead of the monsterproof page #7269

suhr opened this issue Aug 18, 2015 · 22 comments
Labels
I-wrong An incorrect behaviour is observed.

Comments

@suhr
Copy link

suhr commented Aug 18, 2015

Now to reproduce: % ./mach run --release -e https://monsterproof.verigames.com/
OS: Linux x86_64

@suhr suhr changed the title Servo shows gray screen instead of the monsterproof page Servo shows grey screen instead of the monsterproof page Aug 18, 2015
@frewsxcv frewsxcv added the I-wrong An incorrect behaviour is observed. label Aug 19, 2015
@mdinger
Copy link

mdinger commented Jan 22, 2016

I'm seeing this for every page now including the tests/html/about-mozilla.html. Not sure what the issue is. Tested with commit a914727

@mdinger
Copy link

mdinger commented Jan 22, 2016

Bisecting:

Works with 0d1dfb9 (after PR #9210)

@mdinger
Copy link

mdinger commented Jan 22, 2016

Works with e684a20 (after PR #9282)

@mdinger
Copy link

mdinger commented Jan 22, 2016

Works with 1ca2073 (after PR #9348)

@mdinger
Copy link

mdinger commented Jan 22, 2016

Fails with ff09924 (after PR #9366)

@mdinger
Copy link

mdinger commented Jan 22, 2016

Works with d9c6431 (after PR #9359)

@mdinger
Copy link

mdinger commented Jan 22, 2016

Works with 9d2eacf (after PR #9363)

@mdinger
Copy link

mdinger commented Jan 22, 2016

Fails with 74d75c4 (after PR #9364).

The problem started with 74d75c4 (PR #9364)

@jdm
Copy link
Member

jdm commented Jan 23, 2016

...huh. That's a very peculiar result. Could double check that a clobber build (ie. ./mach clean) on 74d75c4 and the revision right before it show the bad/good behaviour respectively?

@mdinger
Copy link

mdinger commented Jan 23, 2016

What I was doing was:

cd servo
git checkout -b new_name hash
sudo rm -r *
git reset --hard
./mach build --dev

The delete was with a gui because I wasn't sure if the command line would also remove the hidden files but that is basically what I did. Seems like that would be just as good as what you're suggesting.

@jdm
Copy link
Member

jdm commented Jan 23, 2016

I agree. Wow.

@jdm
Copy link
Member

jdm commented Jan 23, 2016

@glennw @mrobinson Any suggestions for investigating how a libc update caused servo to stop rendering anything?

@glennw
Copy link
Member

glennw commented Jan 23, 2016

Does using -o mode to output an image work?

@mdinger
Copy link

mdinger commented Jan 23, 2016

You mean like ./mach run -o www.google.com or like something else?

EDIT: I mean, I can check what the mach commands are next time I have linux up but until then, I'm just guessing.

@glennw
Copy link
Member

glennw commented Jan 23, 2016

Try ./mach run https://google.com -o output.png

@glennw
Copy link
Member

glennw commented Jan 23, 2016

(Or, perhaps try that with tests/html/about-mozilla.html first)

@mdinger
Copy link

mdinger commented Jan 23, 2016

-o output.png works properly with both tests/html/about-mozilla.html and google.com but normal running just generates a blank grey screen

@mbrubeck
Copy link
Contributor

Works with 9d2eacf (after PR #9363)
Fails with 74d75c4 (after PR #9364).

The problem started with 74d75c4 (PR #9364)

That regression range also includes PR #9339. My testing shows that the first bad revision is actually f5c6c60 (#9339).

@mbrubeck
Copy link
Contributor

Filed #9451 for this recent Linux-specific regression.

@suhr
Copy link
Author

suhr commented Jan 28, 2016

I guess this can be closed in favour of #9451?

@jdm
Copy link
Member

jdm commented Jan 28, 2016

Seems reasonable.

@jdm jdm closed this as completed Jan 28, 2016
@mdinger
Copy link

mdinger commented Jan 29, 2016

Thanks @mbrubeck. I was actually trying to follow the bors PR merge order but apparently I failed. I did manually compare the branches and they appeared to only be a single commit apart. Comparing the servo tree is actually quite tricky because of the volume of commits and merges which occur. I'm glad you pointed out that trick!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-wrong An incorrect behaviour is observed.
Projects
None yet
Development

No branches or pull requests

6 participants