Closed Bug 684107 Opened 13 years ago Closed 13 years ago

Remove private browsing code from DownloadLastDir.jsm after bug 679784 is fixed

Categories

(Toolkit :: Preferences, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

Attachments

(2 files)

I'll fix this once bug 679784 lands on m-c, unless someone beats me to it.
Attached patch patchSplinter Review
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #558054 - Flags: review?(ehsan)
Comment on attachment 558054 [details] [diff] [review]
patch

Looks great!
Attachment #558054 - Flags: review?(ehsan) → review+
http://hg.mozilla.org/mozilla-central/rev/e80b7fe1c11c
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
looks like this forgot to remove the import() for Dict.jsm.
(In reply to Gavin Sharp (use gavin@gavinsharp.com for email) from comment #5)
> looks like this forgot to remove the import() for Dict.jsm.

And XPCOMUtils.jsm, which is interesting because it wasn't ever used here. :/
Attachment #590117 - Flags: review?(gavin.sharp)
Attachment #590117 - Flags: review?(gavin.sharp) → review+
You need to log in before you can comment on or make changes to this bug.