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.

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

(Github Comment Enhancer) Serious performance issues in Firefox #97

Closed
thomaslevesque opened this issue Jan 25, 2016 · 10 comments
Closed

Comments

@thomaslevesque
Copy link

I was wondering why Firefox always froze when I opened a PR in GitHub, and I realized that it worked fine when I disabled GreaseMonkey. Since GCE is the only user script I installed, I'm afraid it's the culprit...

Note that it works fine in Chrome.

@jerone
Copy link
Owner

jerone commented Jan 25, 2016

Hi. I'm sorry to hear you've got problems with this userscript. I can't confirm any performance issues as I just created a few PR's in the last hour without problem. Can you give me some more information, beginning with version of this userscript, FF, GM...

Also can you check if any related errors appear in the Browser Console (Menu > Developer > Browser Console).

@thomaslevesque
Copy link
Author

Hi,

I'm using FF 43.0.4, GM 3.6 and GCE 2.8.3. Here's an example of a PR for which the browser freezes temporarily: FakeItEasy/FakeItEasy#574. In general, the larger the PR, the longer the freeze, and that one is pretty big.

Here's the console output:

mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create frameworks-9ee55ceaf87fc34dc86334249fef6cbece88e815478e0fbe81642d57ed0fff89.js:1:20147
Empty string passed to getElementById(). github-394e52ce754d3bfd2289c98e49302efbad2b1ddb5c40d88f0b4b68088f2a74cd.js:6:10437
Firefox can't establish a connection to the server at wss://live.github.com/_sockets/MTg2NDI4MTo3OGZjMDE4OTYxZjVjMjhkMjg2NTU0YzFiMzM3OTg5YjoyZWNhMDQ2MmQ1MDNhZGE0YmZhYjZlYjQ4NTJkMDk3MTdjNGRhYzZmMTA1Y2Q0NWI3ZDJkMTkwZTBlMWVhYjRi--2ca2126eb698cbd1730175f9ddda7d4dbc5c1be6. github-394e52ce754d3bfd2289c98e49302efbad2b1ddb5c40d88f0b4b68088f2a74cd.js:5:25141
The connection to wss://live.github.com/_sockets/MTg2NDI4MTo3OGZjMDE4OTYxZjVjMjhkMjg2NTU0YzFiMzM3OTg5YjoyZWNhMDQ2MmQ1MDNhZGE0YmZhYjZlYjQ4NTJkMDk3MTdjNGRhYzZmMTA1Y2Q0NWI3ZDJkMTkwZTBlMWVhYjRi--2ca2126eb698cbd1730175f9ddda7d4dbc5c1be6 was interrupted while the page was loading. github-394e52ce754d3bfd2289c98e49302efbad2b1ddb5c40d88f0b4b68088f2a74cd.js:5:25141

@jerone
Copy link
Owner

jerone commented Jan 26, 2016

Here's the console output:

Those errors seem unrelated to GCE.

Here's an example of a PR for which the browser freezes temporarily: FakeItEasy/FakeItEasy#574.

And when do you have problems with freezes?:

  1. When navigating to this PR from another Github page?
  2. When navigating to this PR in a new tab (no history)?
  3. When navigating in this PR from Conversation to Commits?
  4. When navigating in this PR from Conversation to Files changed?

@thomaslevesque
Copy link
Author

Every time I navigate to the page, whether it's in a new tab or when navigating from another page. Switching to the Files changed tab is even worse, sometimes freezing for more than 10s and showing the "long running script" dialog

@jerone
Copy link
Owner

jerone commented Jan 27, 2016

This might be related to changes to the website: #98

@thomaslevesque
Copy link
Author

This might be related to changes to the website: #98

I don't think so; I noticed this problem a while ago, before Github introduced its toolbar.

@mzso
Copy link

mzso commented Apr 23, 2016

Hmmm I re-enabled this addon to make a comment, but now it doesn't seem to work at all. None of the features seem to appear. eg: I can't quote with a reply.
By the way I'm on aurora and have e10s enabled.
(PS I opened #104 )

@jerone
Copy link
Owner

jerone commented Apr 23, 2016

Yeah it appears that Github changed their site again, completely breaking this (and other) userscript.

@thomaslevesque
Copy link
Author

Yeah it appears that Github changed their site again, completely breaking this (and other) userscript.

I can confirm. I had been using my own userscript for a while (it just mapped custom keyboard shortcuts to the standard Github formatting buttons), and now it's broken as well.

@jerone
Copy link
Owner

jerone commented Apr 27, 2016

Closing in favor of #106

@jerone jerone closed this as completed Apr 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants