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

Already on GitHub? Sign in to your account

Example webpages that cause phantom to "spin" forever and consume tons of memory #11189

Closed
dknecht opened this Issue Mar 29, 2013 · 15 comments

Comments

Projects
None yet
5 participants

dknecht commented Mar 29, 2013

console.log('Loading a web page');
var page = require('webpage').create();
var url = 'http://www.animalasia.com.hk/';
page.open(url, function (status) {
    //Page is loaded!
    phantom.exit();
});

http://rapture.romangrey.net
http://mocapanalytics.com

Collaborator

JamesMGreene commented Mar 29, 2013

OS? PhantomJS version? Binary or compiled from source? curl -v output for each of those sites?

dknecht commented Mar 29, 2013

very close to 1.9. I have tried it on multiple platforms and multiple version. If you want to see it happen i would just run sample code posted above.

dknecht commented Mar 29, 2013

Two more sites you can try...

timesofassam.com
amma.org.au

Owner

ariya commented Mar 31, 2013

I can reproduce this.

Collaborator

Vitallium commented Mar 31, 2013

Well... I can reproduce it on Windows too. But PhantomJS crashes sometimes =\

Owner

ariya commented Mar 31, 2013

That's correct, I also see the crash from time to time.

dknecht commented Mar 31, 2013

Thanks for looking at. Let us know if you need any more details.

dknecht commented Apr 1, 2013

Another site.

lideres.hispanicaccess.org

Do you want me to keep adding these?

Collaborator

JamesMGreene commented Apr 1, 2013

@dknecht: I would say that there isn't any need for you to add more at this point. Give us (or some other willing contributor!) a chance to fix the issue we're seeing with these sites first, then if you are still finding additional sites with issues post-fix, you can certainly file them in another new issue.

Thanks!

Collaborator

Vitallium commented Apr 1, 2013

Actually, I've already found what is causing this issue. It's a Webkit bug. More specifically: bug is in mixing async and sync resource loading.

dknecht commented Apr 1, 2013

Something that can be fixed in PhantomJS?

Collaborator

Vitallium commented Apr 2, 2013

Ok. I've fixed the crash and all urls are working with my patch.
But, the patch is too dangerous, so I need to perform some tests before opening a pull request (if it will be accepted ever ;))
Patch: https://gist.github.com/Vitallium/5294430

Collaborator

JamesMGreene commented Apr 3, 2013

@Vitallium:

  1. I added a few comments on your patch, please check them out.
  2. Your patch is actually a desired feature that has been mentioned several other times, see:
  3. I've filed a new issue, #11198, to properly request this feature.

http://www.nexus-web.net/interview/champagne/

spins in WebCore::RenderBlock::logicalLeftOffsetForLine
possible forever

@Vitallium Vitallium was assigned May 9, 2013

Collaborator

Vitallium commented Jul 31, 2014

Fixed in 2.0

@Vitallium Vitallium closed this Jul 31, 2014

@JamesMGreene JamesMGreene assigned Vitallium and unassigned Vitallium Jul 31, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment