Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

JS iloop policing needs to be enhanced #7973

Closed
alexwykoff opened this issue Mar 29, 2017 · 2 comments
Closed

JS iloop policing needs to be enhanced #7973

alexwykoff opened this issue Mar 29, 2017 · 2 comments

Comments

@alexwykoff
Copy link
Contributor

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    When visiting https://www.extremetech.com/internet/220696-forbes-forces-readers-to-turn-off-ad-blockers-promptly-serves-malware the CPU spikes and holds for a considerably long period of time.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Tested on OS X

  • Brave Version (revision SHA):
    0.14.0 RC2 (b00c5d9)

  • Steps to reproduce:

    1. Visit b00c5d9
    2. Check CPU usage
  • Actual result:
    The page should load quickly and CPU usage should be nominal (certainly not 100% for more than a couple seconds)

  • Expected result:
    The CPU usage stays above 100% (for multicore systems) for a considerably long time

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    0.13.5 appears to behave better

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:

@alexwykoff alexwykoff added this to the 0.14.1 milestone Mar 29, 2017
@alexwykoff alexwykoff changed the title An extremetech page is constantly pegging CPU JS iloop policing needs to be enhanced Mar 29, 2017
@bsclifton
Copy link
Member

Moving to 0.14.3

@bsclifton bsclifton modified the milestones: 0.14.3, 0.14.2 Apr 4, 2017
@bsclifton bsclifton added needs-owner ♞ This issue is tagged for an upcoming release but has no owner. and removed needs-owner ♞ This issue is tagged for an upcoming release but has no owner. labels Apr 7, 2017
@alexwykoff alexwykoff modified the milestones: Backlog, 0.15.1 Apr 18, 2017
@alexwykoff
Copy link
Contributor Author

Can't reproduce on latest, but if we see it again, we can pull it into a near milestone

@bbondy bbondy removed this from the Backlog milestone Oct 19, 2017
@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@bsclifton bsclifton added stale and removed needs-owner ♞ This issue is tagged for an upcoming release but has no owner. labels Aug 27, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Aug 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants