Closed
Bug 675858
Opened 10 years ago
Closed 10 years ago
Error: Services is not defined chrome://navigator/content/pageinfo/permissions.js
Categories
(SeaMonkey :: Page Info, defect)
SeaMonkey
Page Info
Tracking
(seamonkey2.4 fixed, seamonkey2.5 fixed, seamonkey2.6 fixed)
RESOLVED
FIXED
seamonkey2.6
People
(Reporter: philip.chee, Assigned: stanio)
References
Details
(Keywords: regression, Whiteboard: [good first bug])
Attachments
(1 file)
1.90 KB,
patch
|
Callek
:
review+
Callek
:
approval-comm-aurora+
Callek
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
Error: Services is not defined Source file: chrome://navigator/content/pageinfo/permissions.js Line: 38 In pageInfo.xul I think utilityOverlay.js should come before any calls to Services.
![]() |
Reporter | |
Updated•10 years ago
|
Whiteboard: [good first bug]
Assignee | ||
Comment 1•10 years ago
|
||
Per comment #0 moved the inclusion of utilityOverlay.js before any calls to Services. I've identified this should be just prior pageInfo.js. It appears to fix the issue for me (no errors in the Error Console) and the Live HTTP Headers extension tab in the Page Info now shows the headers just fine.
Attachment #556301 -
Flags: review?(db48x)
Comment 2•10 years ago
|
||
Comment on attachment 556301 [details] [diff] [review] Suggested patch I have not verified that this fixes the bug, but I do know this won't break anything, so lets call it good, and VERIFY once this lands.
Attachment #556301 -
Flags: review?(db48x)
Attachment #556301 -
Flags: review+
Attachment #556301 -
Flags: approval-comm-beta+
Attachment #556301 -
Flags: approval-comm-aurora+
![]() |
Reporter | |
Updated•10 years ago
|
Keywords: checkin-needed
![]() |
Reporter | |
Updated•10 years ago
|
Assignee: nobody → stanio
![]() |
Reporter | |
Comment 3•10 years ago
|
||
Pushed: http://hg.mozilla.org/comm-central/rev/5d1bb791ad9d http://hg.mozilla.org/releases/comm-aurora/rev/06cd5532ae61 http://hg.mozilla.org/releases/comm-beta/rev/63c6090eb921 Thank you Stanimir! Sorry I missed this one.
Status: NEW → RESOLVED
Closed: 10 years ago
status-seamonkey2.4:
--- → fixed
status-seamonkey2.5:
--- → fixed
status-seamonkey2.6:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.6
![]() |
Reporter | |
Updated•10 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•