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

50% CPU usage #46

Closed
fcapano opened this issue Oct 11, 2012 · 7 comments
Closed

50% CPU usage #46

fcapano opened this issue Oct 11, 2012 · 7 comments

Comments

@fcapano
Copy link
Member

fcapano commented Oct 11, 2012

Original author: filar...@gmail.com (September 07, 2012 07:31:56)

Hello,
I've noticed that since last updates, ETC often brings up the CPU usage to 50-60% (I can see this from Chrome Task Manager).

I have been forced to disable it otherwise my PC becomes very slow.
I'm running 21.0.1180.89 and Windows 7 SP1 32-bit

Original issue: http://code.google.com/p/editthiscookie/issues/detail?id=47

@fcapano
Copy link
Member Author

fcapano commented Oct 11, 2012

From fcap...@gmail.com on September 07, 2012 10:03:01
Hi, sorry to hear that. Does it happen when you open the popup/options page, or also right after opening chrome browser?

@fcapano
Copy link
Member Author

fcapano commented Oct 11, 2012

From fcap...@gmail.com on September 07, 2012 10:08:29
Can you also tell me which version of Edit This Cookie are you using? There ave been different updates yesterday, so please check that you are using the latest version (1.1.10)

@fcapano
Copy link
Member Author

fcapano commented Oct 11, 2012

From fcap...@gmail.com on September 15, 2012 16:48:52
The issue should be fixed in version 1.1.20. Let me know if you still have problems!

@fcapano
Copy link
Member Author

fcapano commented Oct 11, 2012

From FEMA38...@gmail.com on September 17, 2012 23:53:40
I am using 1.1.21 and it is consistently taking 28-33% CPU, according to Chrome Task Manager. It only seems to happen after chrome has been open for a while.
Thanks

@fcapano
Copy link
Member Author

fcapano commented Oct 11, 2012

From fcap...@gmail.com on September 18, 2012 09:49:32
Do you have any cookies blocked or protected?
Are you using the option to shorten the cookies' maximum age?
Do you have any extension's page open when the issue appears?

Could you do the following while the extension's cpu usage is high?:

  • Go to "chrome://chrome/extensions/"
  • Ensure "Developer mode" is enabled
  • Go to "Edit This Cookie"
  • Click on "_generated_background_page.html"
  • Go to console
  • Copy and paste here
  • Back to the console, Right-click -> Clear console
  • Does it stay empty or new messages appear?

Thank you for your time, hopefully this time I'll be able to fix the issue for real

@JonasNo
Copy link

JonasNo commented May 2, 2015

This function gets into an infinite loop:
background.js line 60 -> "chrome.cookies.onChanged.addListener( function(changeInfo) {"

changeinfo:
cause: "overwrite"
cookie: Object
removed: true

The function "chrome.cookies.get" at line 84 executes the callback function with currentCookie = null.
That cause an exception in "compareCookies" line 89.
"cookie_helpers.js:99 Cannot read property 'name' of null"

The function "compareCookies" returns false after the exception. Than calls "chrome.cookies.set(newCookie);" at line 92 in background.js

This causes an infinite loop.

"chrome.cookies.set" cause "chrome.cookies.onChanged" to be called and it repeats.

The infinite loop starts somehow when chrome is idle for a period of time.

@fcapano
Copy link
Member Author

fcapano commented Apr 27, 2017

I don't think there's been any report regarding this in a long time, I'll close this issue.

@fcapano fcapano closed this as completed Apr 27, 2017
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

2 participants