402244 - Page Info broken when opened from sidebar

Status: RESOLVED FIXED

(Firefox :: Page Info Window, defect)

Description

[Florian Quèze [:florian]]

Attachment: patch v1

Error: window.opener.gBrowser has no properties
Source File: chrome://browser/content/pageinfo/security.js
Line: 73

Comment 1

[Asaf Romano (gone)]

Comment on attachment 287127 [details] [diff] [review]
patch v1

hrm, sp cannot be null if ui isn't (queryinterface throws). I kinda prefer 
ui.QueryInterface(nsISSLStatusProvider);
status = ui.SSLStatus;

looks good otherwise.

Comment 2

[Florian Quèze [:florian]]

Attachment: patch v2

Comment 3

[Reed Loden [:reed]]

Checking in browser/base/content/pageinfo/security.js;
/cvsroot/mozilla/browser/base/content/pageinfo/security.js,v  <--  security.js
new revision: 1.9; previous revision: 1.8
done

Comment 4

[Florian Quèze [:florian]]

Attachment: seamonkey port

Comment 5

[Daniel Brooks [:db48x]]

Comment on attachment 293248 [details] [diff] [review]
seamonkey port

r=db48x

Comment 6

[[e-mail]]

So what's the bug in suite here? As far as I can tell, view page info for a sidebar tab returns security status for the content area, and this patch makes view frame info do that too, which is incorrect as per bug 138479.

Comment 7

[[e-mail]]

Comment on attachment 293248 [details] [diff] [review]
seamonkey port

I think we need a separate bug filed for suite.