Thanks to visit codestin.com
Credit goes to github.com

Skip to content

perf(initialisation): Remove the timeout call in init. #223

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

Merged
merged 1 commit into from
Jan 15, 2016
Merged

Conversation

ValentinH
Copy link
Member

Need tests on all browsers to ensure there is no issue.
Test page: http://jsfiddle.net/o025tpr1/embedded/result/
As mentioned in #220.

Validated browsers (please comment if you have a tested successfully a browser):

  • Chrome 47 OSX
  • Safari 9 OSX
  • Firefox 42/43 OSX
  • Chrome Ubuntu
  • Firefox Ubuntu
  • Chrome Windows
  • Firefox Windows
  • IE 10
  • IE 11
  • MS Edge
  • Chrome Android
  • Firefox Android
  • Opera Android
  • Chrome iOS
  • Safari iOS

@codecov-io
Copy link

Current coverage is 100.00%

Merging #223 into master will not affect coverage as of 571fde3

@@            master    #223   diff @@
======================================
  Files            1       1       
  Stmts          622     621     -1
  Branches         0       0       
  Methods          0       0       
======================================
- Hit            622     621     -1
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of 571fde3

Powered by Codecov. Updated on successful CI builds.

@elifer5000
Copy link

Tested in:
Chrome Windows
IE 11
MS Edge

Didn't see any issues

@ValentinH
Copy link
Member Author

Thanks, I update the list

ValentinH added a commit that referenced this pull request Jan 15, 2016
perf(initialisation): Remove the timeout call in init.
@ValentinH ValentinH merged commit dbaddf7 into master Jan 15, 2016
@ValentinH ValentinH deleted the no-timeout branch January 24, 2016 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants