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.

Sign up for GitHub

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

Jump to bottom

Implement timers (setTimeout/setInterval) in workers #3236

Closed
jdm opened this issue Sep 7, 2014 · 2 comments
Closed

Implement timers (setTimeout/setInterval) in workers #3236

jdm opened this issue Sep 7, 2014 · 2 comments
Labels
A-content/dom Interacting with the DOM from web content E-more-complex Variable effort required; may require a mentor. Recommended solution is clearly described in the iss

Comments

@jdm
Copy link
Member

jdm commented Sep 7, 2014

We should be able to share the current timer code that exists in dom/window.rs. XMLHttpRequest/xmlhttprequest-timeout-worker-twice.html is one example of a web-platform-test that requires this.

Spec: http://www.whatwg.org/specs/web-apps/current-work/multipage/workers.html#apis-available-to-workers (http://www.whatwg.org/specs/web-apps/current-work/multipage/webappapis.html#windowtimers)
@jdm jdm added A-content/dom Interacting with the DOM from web content E-more-complex Variable effort required; may require a mentor. Recommended solution is clearly described in the iss labels Sep 7, 2014
@jdm
Copy link
Member Author

jdm commented Sep 7, 2014

Also XMLHttpRequest/xmlhttprequest-timeout-worker-overridesexpires.html.

mukilan added a commit to mukilan/servo that referenced this issue Oct 13, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
3c11d37
mukilan added a commit to mukilan/servo that referenced this issue Oct 13, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
097ea7d
@mukilan mukilan mentioned this issue Oct 13, 2014
Merged
mukilan added a commit to mukilan/servo that referenced this issue Oct 14, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
d6d66bf
mukilan added a commit to mukilan/servo that referenced this issue Oct 14, 2014
@mukilan
fixup! Issue servo#3236 - Implement timers (setTimeout/setInterval) f…
37a8ac8 
…or workers
mukilan added a commit to mukilan/servo that referenced this issue Oct 14, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
b7ef638
bors-servo pushed a commit that referenced this issue Oct 15, 2014
auto merge of #3662 : mukilan/servo/worker_timer, r=jdm
1cb6bde 
Closes issue #3236
mukilan added a commit to mukilan/servo that referenced this issue Oct 15, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
2812b7d
mukilan added a commit to mukilan/servo that referenced this issue Oct 15, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
10ddce2
bors-servo pushed a commit that referenced this issue Oct 15, 2014
auto merge of #3662 : mukilan/servo/worker_timer, r=jdm
5741436 
Closes issue #3236
bors-servo pushed a commit that referenced this issue Oct 15, 2014
auto merge of #3662 : mukilan/servo/worker_timer, r=jdm
8b4164b 
Closes issue #3236
bors-servo pushed a commit that referenced this issue Oct 15, 2014
auto merge of #3662 : mukilan/servo/worker_timer, r=jdm
696b6de 
Closes issue #3236
bors-servo pushed a commit that referenced this issue Oct 15, 2014
auto merge of #3662 : mukilan/servo/worker_timer, r=jdm
e6ef1e1 
Closes issue #3236
mukilan added a commit to mukilan/servo that referenced this issue Oct 16, 2014
@mukilan
Issue servo#3236 - Implement timers (setTimeout/setInterval) for workers
0d7e13f
bors-servo pushed a commit that referenced this issue Oct 16, 2014
auto merge of #3662 : mukilan/servo/worker_timer, r=jdm
f94228d 
Closes issue #3236
@Ms2ger
Copy link
Contributor

Ms2ger commented Oct 30, 2014

Fixed by #3662.

@Ms2ger Ms2ger closed this as completed Oct 30, 2014
@mukilan mukilan mentioned this issue Nov 4, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-content/dom Interacting with the DOM from web content E-more-complex Variable effort required; may require a mentor. Recommended solution is clearly described in the iss
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jdm @Ms2ger