-
Notifications
You must be signed in to change notification settings - Fork 62
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
Fix issue #49 #50
Fix issue #49 #50
Conversation
@calvinmetcalf please take a look |
looking at it now |
So this is not an appropriate fix? Is it maybe good enough or is there a On Tuesday, September 8, 2015, Calvin Metcalf notifications@github.com
|
adding a 4 to this line might fix it too, but this will probably as well, I'd like to understand how we can get into this situation in the first place because it sounds like setTimeout is calling process.nextTick here somehow |
I managed to reproduce it by opening a new window with Just tried setting |
@joeduncan fascinating are you able to write a gist? |
@calvinmetcalf i just did some more debugging, it does not seem to have anything to do with opening a new window (except for maybe slowing it down). It seems that Chrome does not execute |
So the idea is that if there is an error thrown by one of the functions I don't understand how drainQueue can pause and the timeout be called in On Tue, Sep 8, 2015, 4:30 PM Joe Duncan notifications@github.com wrote:
|
I created a short example (with fluxible) to reproduce error in Firefox, https://github.com/joeduncan/node-process-example
http://localhost:3000/ |
excellent thank you testing |
So what is the final verdict here folks? Is this patchup the solution or will someone be researching a core issue fix here to avoid racy problems from coming up? |
I'm trying to reproduce now |
the following triggers it <html>
<head>
<meta charSet="utf-8" />
<title>title</title>
<meta name="viewport" content="width=device-width, user-scalable=no" />
<link rel="stylesheet" href="http://yui.yahooapis.com/pure/0.5.0/pure-min.css" />
</head>
<body>
</body>
<script>
var called = false;
setTimeout(function (){
console.log('should be true', called);
});
console.log('opening');
window.open('http://example.com/some/url', 'title', 'toolbar=no, location=no, directories=no, status=no, menubar=no, scrollbars=no, resizable=no, copyhistory=no, width=100, height=200, top=20, left=20');
console.log('opened');
called = true;
</script>
</html> |
opened an issue with firefox https://bugzilla.mozilla.org/show_bug.cgi?id=1202918 |
So this ends up being a browser issue? Do we need to patch it? Seems like On Tuesday, September 8, 2015, Calvin Metcalf notifications@github.com
|
may as well merge it, but yeah it seems to be a firefox bug |
thanks for the work folks! merged and released in v0.11.2 |
Fixes issue #49
Unless there is a better way to fix it ;)