Permanent leak in browser_pageInfo.js

RESOLVED FIXED in Firefox 21

Status

()

Firefox
Page Info Window
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: emorley, Assigned: keeler)

Tracking

({memory-leak})

Trunk
Firefox 21
memory-leak
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [test disabled])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=12959284&tree=Mozilla-Inbound

[browser/base/content/test/browser_pageInfo.js]
  1 window(s) [url = about:blank]
  1 window(s) [url = chrome://browser/content/pageinfo/pageInfo.xul]
(Reporter)

Comment 1

6 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=12959805&tree=Mozilla-Inbound

[browser/base/content/test/browser_pageInfo.js]
  1 window(s) [url = about:blank]
  1 window(s) [url = chrome://browser/content/pageinfo/pageInfo.xul]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Is this really intermittent? According to my bugmail this occurred every time we've hit the threshold since bug 772388 landed.
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 23

6 years ago
(In reply to Dão Gottwald [:dao] from comment #19)
> Is this really intermittent? According to my bugmail this occurred every
> time we've hit the threshold since bug 772388 landed.

I checked a few other logs where we hadn't gone over the threshold and I couldn't find one where it wasn't present.
Summary: Intermittent leak in browser_pageInfo.js → Permanent leak in browser_pageInfo.js
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
http://hg.mozilla.org/integration/mozilla-inbound/rev/1fd41677351a
Blocks: 458579
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [orange] → [orange][test disabled][leave open]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Updated

6 years ago
No longer blocks: 438871
Whiteboard: [orange][test disabled][leave open] → [test disabled]
Comment hidden (Treeherder Robot)
(Assignee)

Comment 61

5 years ago
I fixed this leak in bug 839193:
https://tbpl.mozilla.org/?tree=Try&rev=745b58bcb840
Depends on: 839193
(Assignee)

Comment 62

5 years ago
Created attachment 712641 [details] [diff] [review]
patch to re-enable test
Attachment #712641 - Flags: review?(dao)

Updated

5 years ago
Attachment #712641 - Flags: review?(dao) → review+
https://hg.mozilla.org/mozilla-central/rev/8d656a3f2261
Assignee: nobody → dkeeler
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 21
You need to log in before you can comment on or make changes to this bug.