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

Revert httpd.js to the revision not causing FirefoxDriver performance degradation #8557

Closed
lukeis opened this Issue Mar 4, 2016 · 7 comments

Comments

Projects
None yet
1 participant
@lukeis
Member

lukeis commented Mar 4, 2016

Originally reported on Google Code with ID 8557

Since selenium 2.45, updated httpd.js caused performance degradation
with FirefoxDriver across all the platforms. The problem seems to
start happening since 5db95a224bc6 (http://hg.mozilla.org/mozilla-central/rev/5db95a224bc6).
This reverts httpd.js to the revision which doesn't have this problem (http://hg.mozilla.org/mozilla-central/rev/d3ba16c3c8a0)
though the issue probably should still be investigated.

For instance, single spec at watir-webdriver is running 50 seconds with httpd.js from
http://hg.mozilla.org/mozilla-central/rev/5db95a224bc6 and 7 seconds with http://hg.mozilla.org/mozilla-central/rev/d3ba16c3c8a0
(revision we revert httpd.js to).

Reported by AlexR@toptal.com on 2015-03-03 09:33:48


- _Attachment: [0001-Revert-httpd.js-to-the-revision-not-causing-performa.patch](https://storage.googleapis.com/google-code-attachments/selenium/issue-8557/comment-0/0001-Revert-httpd.js-to-the-revision-not-causing-performa.patch)_
@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member
By the way, CLA has been signed.

Reported by AlexR@toptal.com on 2015-03-03 09:35:42

Member

lukeis commented Mar 4, 2016

By the way, CLA has been signed.

Reported by AlexR@toptal.com on 2015-03-03 09:35:42

@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member
can you run the test against the latest in selenium master to see if it is fixed?

Otherwise can you show me how to run a spec that hits this issue?

Reported by luke.semerau on 2015-03-03 18:53:04

Member

lukeis commented Mar 4, 2016

can you run the test against the latest in selenium master to see if it is fixed?

Otherwise can you show me how to run a spec that hits this issue?

Reported by luke.semerau on 2015-03-03 18:53:04

@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member
Also please make sure this httpd.js version is Firefox 36 compatible.

Reported by barancev on 2015-03-03 20:17:50

  • Labels added: Browser-Firefox
Member

lukeis commented Mar 4, 2016

Also please make sure this httpd.js version is Firefox 36 compatible.

Reported by barancev on 2015-03-03 20:17:50

  • Labels added: Browser-Firefox
@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member

Reported by barancev on 2015-03-05 22:05:55

  • Status changed: NeedsClarification
Member

lukeis commented Mar 4, 2016

Reported by barancev on 2015-03-05 22:05:55

  • Status changed: NeedsClarification
@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member
For the record, I'm just setting things up to try and test all supported Firefox versions.

Reported by AlexR@toptal.com on 2015-03-06 05:17:52

Member

lukeis commented Mar 4, 2016

For the record, I'm just setting things up to try and test all supported Firefox versions.

Reported by AlexR@toptal.com on 2015-03-06 05:17:52

@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member
this was fixed by 4af80cca9131a345b935d6ae76e14f8bebabf6fe

Reported by luke.semerau on 2015-03-12 16:19:38

  • Status changed: Fixed
Member

lukeis commented Mar 4, 2016

this was fixed by 4af80cca9131a345b935d6ae76e14f8bebabf6fe

Reported by luke.semerau on 2015-03-12 16:19:38

  • Status changed: Fixed
@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Mar 4, 2016

Member

Reported by luke.semerau on 2015-09-17 18:25:55

  • Labels added: Restrict-AddIssueComment-Commit
Member

lukeis commented Mar 4, 2016

Reported by luke.semerau on 2015-09-17 18:25:55

  • Labels added: Restrict-AddIssueComment-Commit

@lukeis lukeis closed this Mar 4, 2016

@SeleniumHQ SeleniumHQ locked and limited conversation to collaborators Mar 4, 2016

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