Last Comment Bug 724222 - View page info -> Security: doesn't show cookies were set for sites with ip:8080 address (see Firefox Bug 409174).
: View page info -> Security: doesn't show cookies were set for sites with ip:8...
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: Page Info (show other bugs)
: Trunk
: All All
: -- normal (vote)
: seamonkey2.10
Assigned To: Philip Chee
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-04 02:41 PST by Philip Chee
Modified: 2012-02-06 20:33 PST (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
fixed


Attachments
Patch v1.0 (3.38 KB, patch)
2012-02-04 02:51 PST, Philip Chee
iann_bugzilla: review+
Details | Diff | Splinter Review

Description Philip Chee 2012-02-04 02:41:09 PST
q.v.
Bug 409174: View page info -> Security: doesn't show cookies were set for sites with ip:8080 address. 2008
  (Regression) Bug 435577 - Security tab broken for moz-icon or jar URIs.
Comment 1 Philip Chee 2012-02-04 02:51:31 PST
Created attachment 594414 [details] [diff] [review]
Patch v1.0

Tested with:
http://www.mozdev.org:8080/
and
moz-icon://.pdf?size=16

> +function hostHasCookies(uri) {
> +  var hostName;
> +  try {
> +    hostName = uri.asciiHost;
> +  }
> +  catch (e) {
> +  }

uri.asciiHost will throw for moz-icon:
Comment 2 Ian Neal 2012-02-06 14:44:50 PST
Comment on attachment 594414 [details] [diff] [review]
Patch v1.0

In theory I should ask you to change the function arguments to aUri
r=me either way.
Comment 3 Philip Chee 2012-02-06 20:33:48 PST
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/9ff442d02f87

Note You need to log in before you can comment on or make changes to this bug.