gorhill / uMatrix

uMatrix: Point and click matrix to filter net requests according to source, destination and type
GNU General Public License v3.0
4.55k stars 470 forks source link

Some urls stay in the drop down list forever in detailed statistics #41

Closed my-password-is-password closed 9 years ago

my-password-is-password commented 9 years ago

Use the browser normally opening links in a new tab and closing them when you're done. Then check the drop down list in Statistics after some time. Some urls stay in the list forever even though the tab has been closed a long time ago.

gorhill commented 9 years ago

When you say "after some time", what do you mean exactly?

my-password-is-password commented 9 years ago

Like any time after it checks if the web page should flush the stats.

gorhill commented 9 years ago

When I closed the web pages, they disappears from the drop down menu after 10 seconds. I am going to need more details to repro.

my-password-is-password commented 9 years ago

I have a quick way to get urls stuck in the list but I don't know if the quick way is part of the problem.

  1. Go to wired.com
  2. Open like 10+ links in a new tab
  3. On the wired.com tab, right click and do a "Close tabs to the right"
  4. Check the drop down in Statistics
gorhill commented 9 years ago

Works all fine here.

Browser? version?

my-password-is-password commented 9 years ago

Stable Chrome 38.0.2125.111 m

This is a screenshot of my dropdown list after doing the steps above like 3 or 4 times.

umdropdownlist

All tabs are closed and those urls never go away.

gorhill commented 9 years ago

If you enter in the extension console:

Object.keys(µMatrix.pageStats).join('\n');

After you see the over-populated list, what do you get?

my-password-is-password commented 9 years ago

This is what it printed out

"http://chromium-behind-the-scene/
http://chrome-extension-scheme/
http://www.wired.com/2014/11/ikea-bekant-desk/
http://www.wired.com/2014/11/airhopper-hack/
http://www.wired.com/2014/11/oral-history-through-being-cool/
http://www.wired.com/2014/10/virgin-galactics-spaceshiptwo-crashes/
http://www.wired.com/2014/10/max-headroom-interupt/
http://www.wired.com/2014/11/this-week-in-trailers-27/
http://www.wired.com/2014/10/scariest-non-horror-movies/
http://www.wired.com/2014/10/best-week-tv-34/
http://www.wired.com/2014/10/cape-watch-9/
http://chrome-scheme/
http://www.wired.com/2014/11/sneak-peek-adobes-smart-new-ipad-app-designers/
http://www.wired.com/2014/10/navy-self-driving-swarmboats/
http://www.wired.com/category/design
http://www.wired.com/2014/10/hypnotic-video-takes-tripping-worlds-greenhouses/
http://www.wired.com/2014/10/sliders-knobs-dials-give-tablet-physical-interface/
http://www.wired.com/2014/10/bizarre-toy-cameras-heralded-age-instagram/
http://www.wired.com/2014/10/flipboard-3-smartphone-app/
http://www.wired.com/2014/10/mit-media-lab-gets-transforming-logo-courtesy-pentagram/
http://www.wired.com/2014/10/saddest-cubicles/
http://www.wired.com/2014/10/23-beautiful-products-make-waking-suck-little-less/
http://www.nytimes.com/2013/10/27/arts/artsspecial/venturing-beyond-art-to-…ace.html?adxnnl=1&pagewanted=all&adxnnlx=1383070183-I/YzzTLVzDAhRh//EI1fjg
http://www.nytimes.com/glogin?URI=http://www.nytimes.com/2013/10/27/arts/ar…Q235Kk7Q7EzQ24kCDVKfTkQ5BDRz7D7TDCQ245KDQ24Q5BKRuD7!KQ24zDueRQ7DKX!7Q3EQ5C
http://www.nytimes.com/2013/10/27/arts/artsspecial/venturing-beyond-art-to-…tml?adxnnl=1&pagewanted=all&adxnnlx=1383070183-I/YzzTLVzDAhRh//EI1fjg&_r=0
http://jalopnik.com/meet-the-guy-who-drove-across-the-u-s-in-a-record-28-h-…lflow_jalopnik_facebook&utm_source=jalopnik_facebook&utm_medium=socialflow
http://www.wired.com/entertainment/
http://www.wired.com/2014/11/geeks-guide-women-destroy/
http://www.wired.com/2014/10/gamelife-podcast-episode-132/
http://www.wired.com/2014/10/double-action-boogaloo/
http://www.wired.com/2014/10/nintendo-sleep-qol-health/
http://www.wired.com/2014/10/civilization-beyond-earth-review/
http://www.wired.com/2014/10/super-friends-hall-of-justice/
http://www.wired.com/2014/10/sam-max-hit-road-download/
http://www.wired.com/2014/10/mark-mothersbaugh-devo-gallery/
http://www.wired.com/2014/10/binge-guide-fringe/
http://www.wired.com/2014/10/what-to-do-in-washington-dc/
http://www.wired.com/2014/10/marvel-next-5-years-of-movies/
http://arstechnica.com/
http://arstechnica.com/gadgets/2014/11/nexus-9-review-a-premium-tablet-with-a-not-so-premium-feel/
http://arstechnica.com/business/2014/11/from-cash-only-to-nfc-ready-how-we-buy-determines-what-we-buy/
http://arstechnica.com/apple/2014/11/yes-the-5k-retina-imacs-screen-runs-at-60hz-at-5k-resolution/
http://arstechnica.com/information-technology/2014/11/remoteie-gives-free-access-to-internet-explorer-vms-without-the-vm/
http://arstechnica.com/security/2014/11/does-your-phone-company-track-you/
http://arstechnica.com/tech-policy/2014/11/man-sentenced-to-a-year-in-jail-for-firing-laser-50-times-at-police-plane/
http://arstechnica.com/business/2014/11/sprint-dropping-customers-losing-money-and-laying-employees-off/
http://arstechnica.com/security/2014/11/researchers-bridge-air-gap-by-turning-monitors-into-fm-radios/
http://arstechnica.com/tech-policy/2014/11/napster-netflix-founders-top-donors-in-california-voter-politics/
http://arstechnica.com/apple/2014/11/apple-1-computer-sold-out-of-steve-jobs-garage-could-pull-in-600k-or-more/
http://arstechnica.com/security/2014/11/cell-carrier-was-weakest-link-in-hack-of-google-instagram-accounts/
http://arstechnica.com/science/2014/11/ntsb-spaceshiptwo-broke-apart-when-feathering-activated-early/
http://arstechnica.com/gaming/2014/11/hearthstone-bot-maker-closes-shop-after-blizzard-crackdown/
http://arstechnica.com/science/2014/11/as-earth-left-the-last-ice-age-co2-rose-in-fits-and-starts/
http://arstechnica.com/gaming/2014/11/nintendo-president-says-company-must-consider-removing-region-locks/
http://arstechnica.com/business/2014/11/epic-2014-recalling-a-decade-old-imagining-of-the-tech-driven-media-future/
http://arstechnica.com/science/2014/10/one-feared-dead-as-virgin-galactic-spaceshiptwo-destroyed-in-test-flight/
http://arstechnica.com/tech-policy/2014/10/virginia-judge-police-can-demand-a-suspect-unlock-a-phone-with-a-fingerprint/
http://arstechnica.com/security/2014/10/facebook-offers-hidden-service-to-tor-users/
http://arstechnica.com/information-technology/2014/10/consumer-windows-7-preinstalls-come-to-an-end-today/
http://arstechnica.com/apple/2014/10/microsoft-updates-creaky-old-office-2011-for-mac-to-shiny-new-version/
http://arstechnica.com/gadgets/2014/10/andy-rubin-the-founder-of-android-leaves-google/
http://arstechnica.com/gaming/2014/10/updated-numbers-show-ps4-with-at-least-65-percent-of-two-console-market/
http://arstechnica.com/security/2014/10/drupal-sites-had-hours-to-patch-before-attacks-started/
http://arstechnica.com/tech-policy/2014/10/google-ordered-to-pay-a-woman-2250-for-street-view-image-showing-cleavage/
http://arstechnica.com/tech-policy/2014/10/pirate-bay-co-founder-convicted-in-denmarks-largest-hacking-case-ever/
http://arstechnica.com/tech-policy/2014/10/after-massive-danish-hack-gottfrid-svartholm-warg-sentenced-to-3-5-years/
http://arstechnica.com/gadgets/2014/10/a-first-look-at-the-microsoft-band/
http://arstechnica.com/business/2014/10/fcc-reportedly-close-to-reclassifying-isps-as-common-carriers/
http://arstechnica.com/science/2014/10/learn-inflationary-cosmology-from-the-guy-who-invented-it/
http://arstechnica.com/tech-policy/2014/10/verizon-wireless-to-pay-back-customers-allegedly-billed-for-free-calls/
http://arstechnica.com/science/2014/10/in-mice-genetics-dictates-ebola-infection-outcomes/
http://arstechnica.com/gaming/2014/10/zuckerberg-sees-50-to-100-million-oculus-units-sold-in-next-decade/
http://arstechnica.com/tech-policy/2014/10/feds-bust-online-dating-service-that-sent-fake-messages-from-virtual-cupids/
http://arstechnica.com/apple/2014/10/class-action-suit-alleges-2011-apple-macbook-pros-were-defective/
http://chrome-devtools-scheme/"

And this is what the drop down list looks like now.

umdropdownlist2

gorhill commented 9 years ago

Actually, that's not the right dict to look at, that one is longer term. The one used for the lists is:

Object.keys(µMatrix.pageUrlToTabId).join('\n');
my-password-is-password commented 9 years ago
"http://chromium-behind-the-scene/
http://chrome-extension-scheme/
http://www.wired.com/2014/11/ikea-bekant-desk/
http://www.wired.com/2014/11/airhopper-hack/
http://www.wired.com/2014/11/oral-history-through-being-cool/
http://www.wired.com/2014/10/virgin-galactics-spaceshiptwo-crashes/
http://www.wired.com/2014/10/max-headroom-interupt/
http://www.wired.com/2014/11/this-week-in-trailers-27/
http://www.wired.com/2014/10/scariest-non-horror-movies/
http://www.wired.com/2014/10/best-week-tv-34/
http://www.wired.com/2014/10/cape-watch-9/
http://www.wired.com/2014/11/sneak-peek-adobes-smart-new-ipad-app-designers/
http://www.wired.com/2014/10/navy-self-driving-swarmboats/
http://www.nytimes.com/2013/10/27/arts/artsspecial/venturing-beyond-art-to-…tml?adxnnl=1&pagewanted=all&adxnnlx=1383070183-I/YzzTLVzDAhRh//EI1fjg&_r=0
http://jalopnik.com/meet-the-guy-who-drove-across-the-u-s-in-a-record-28-h-…lflow_jalopnik_facebook&utm_source=jalopnik_facebook&utm_medium=socialflow
http://www.wired.com/2014/10/binge-guide-fringe/
http://chrome-devtools-scheme/"

I noticed every now and then when i'm scrolling the drop down list it takes me back to "All". Is that the clean up its doing? Could it be that I'm closing theses tabs while its busy cleaning up?

gorhill commented 9 years ago

There has to be something specific in your environment, I can't reproduce. Entries in that dict above are removed when the function in start.js line 96 is called, whenever a tab is removed.

my-password-is-password commented 9 years ago

No worries, not a big deal anyway.

gorhill commented 9 years ago

not a big deal anyway

Of course it is a big deal. We have to find out why this happens. Can't you breakpoint at start.js/line 102? If I could repro I would do it, but I can't for whatever reason.

my-password-is-password commented 9 years ago

I saw and error but I cleared it and trying to get it again. It was something about the badge and not being able to find the tab with a certain id.

lol. I got blue screen of death doing this over and over.

my-password-is-password commented 9 years ago

They all had ids greater than 0. And the badge error i mentioned earlier had nothing to do with it. I console logged the tabIds on the left.

umdropdownlist3

gorhill commented 9 years ago

The tab ids are fine. I would like to understand why µMatrix.unbindTabFromPageStats in tab.js is not called with these tab ids.

my-password-is-password commented 9 years ago

I think it was one of the flags I was using, chrome://flags/#enable-fast-unload. If not that one, some other flag.

I reset the flags and so far it hasn't been doing it.

Yeah, it was some flag I had changed. Its working good now.

I'm so sorry for wasting your time. :(