Visiting http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-trunk/ in PB mode stores a favicon in the disk cache

VERIFIED FIXED in Firefox 18

Status

()

defect
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: jdm, Assigned: Ehsan)

Tracking

Trunk
mozilla19
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox18+ verified, firefox19+ verified)

Details

(Whiteboard: [testday-20121012])

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

7 years ago
STR:
1. clear disk cache
2. enter PB mode
3. visit http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-trunk/
4. see http://www.mozilla.org/media/img/favicon.png in the disk cache
Reporter

Updated

7 years ago
Assignee

Comment 1

7 years ago
Posted patch Fix (obsolete) — Splinter Review
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #670902 - Flags: review?(josh)
Assignee

Updated

7 years ago
Component: General → Networking: HTTP
Assignee

Updated

7 years ago
Reporter

Updated

7 years ago
Attachment #670902 - Flags: review?(josh) → review+
Reporter

Comment 2

7 years ago
Comment on attachment 670902 [details] [diff] [review]
Fix

My mistake, we need to do this for at least nsBaseChannel as well. I'm looking into the other ones, too.
Attachment #670902 - Flags: review+ → review-
Reporter

Comment 3

7 years ago
Looks like just HTTP and nsBaseChannel require changes.
Assignee

Comment 4

7 years ago
Posted patch Patch (v2)Splinter Review
Attachment #670902 - Attachment is obsolete: true
Attachment #670957 - Flags: review?(josh)
Reporter

Comment 5

7 years ago
Comment on attachment 670957 [details] [diff] [review]
Patch (v2)

I'm going to be safe here and delegate to Jason, since redirects are slippery beasts.
Attachment #670957 - Flags: review?(josh)
Attachment #670957 - Flags: review?(jduell.mcbugs)
Attachment #670957 - Flags: feedback+
Reporter

Updated

7 years ago
Whiteboard: [testday-20121012]
Ehsan, can you please confirm if aurora (18.0a2) is affected as well ?
Assignee

Comment 7

7 years ago
(In reply to bhavana bajaj [:bajaj] from comment #6)
> Ehsan, can you please confirm if aurora (18.0a2) is affected as well ?

It is.  The status-firefox18:affected flag reflects that.

Comment 8

7 years ago
Comment on attachment 670957 [details] [diff] [review]
Patch (v2)

Review of attachment 670957 [details] [diff] [review]:
-----------------------------------------------------------------

::: netwerk/base/src/nsBaseChannel.cpp
@@ +77,5 @@
>    newChannel->SetNotificationCallbacks(mCallbacks);
>    newChannel->SetLoadFlags(mLoadFlags | LOAD_REPLACE);
>  
> +  // Try to preserve the privacy bit if it has been overridden
> +  if (mPrivateBrowsingOverriden) {

AFAICT nsBaseChannel may be dead code--I can't seem to get anything to hit it in gdb at least--but you're right that if we do hit it we need this.

::: netwerk/protocol/http/HttpBaseChannel.cpp
@@ +1542,5 @@
>    newChannel->SetNotificationCallbacks(mCallbacks);
>    newChannel->SetLoadFlags(newLoadFlags);
>  
> +  // Try to preserve the privacy bit if it has been overridden
> +  if (mPrivateBrowsingOverriden) {

I guess this typo is part of the code base now, a la Http referer. Don't want to mess up hg history to fix it.  Correct pronunciation rhymes with "Biden" :)
Attachment #670957 - Flags: review?(jduell.mcbugs) → review+

Comment 9

7 years ago
ahem--make that "nsBaseChannel::Redirect may be dead code"  :)
Assignee

Comment 10

7 years ago
(In reply to Jason Duell (:jduell) from comment #9)
> ahem--make that "nsBaseChannel::Redirect may be dead code"  :)

Wouldn't it be just lovely if nsBaseChannel was dead code?  ;-)

https://hg.mozilla.org/integration/mozilla-inbound/rev/007e45a1f6c9
Assignee

Comment 11

7 years ago
Comment on attachment 670957 [details] [diff] [review]
Patch (v2)

I think this is reasonably low-risk, and we definitely need to take this for Firefox 18.
Attachment #670957 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/007e45a1f6c9
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Attachment #670957 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
No suggestions stored in the disk cache. Verified fixed FF 18b3, Aurora 19.0a2 (2012-12-11).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.