Closed Bug 665276 Opened 13 years ago Closed 13 years ago

Remove nsISHistory_2_0_BRANCH from /suite/common/src/nsSessionStore.js

Categories

(SeaMonkey :: Session Restore, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.4

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

Attachments

(1 file)

Bug 664979 Merged nsISHistory_2_0_BRANCH into nsISHistory. We need to replace all uses of nsISHistory_2_0_BRANCH with plain nsISHistory

http://mxr.mozilla.org/comm-central/search?string=nsISHistory_2_0_BRANCH&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central

Indicates only one use.

See Attachment 540219 [details] [diff]
> -          QueryInterface(Ci.nsISHistory_2_0_BRANCH).reloadCurrentEntry();
> +          QueryInterface(Ci.nsISHistory).reloadCurrentEntry();
Blocks: 667160
Attached patch Patch v1.0Splinter Review
Should fix Bug 667160 as well.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #541939 - Flags: review?(neil)
Comment on attachment 541939 [details] [diff] [review]
Patch v1.0

Ah yes, in bug 639771 comment #1 I suggested we make it easier to delete, preventing us from accidentally running into bug 664979 comment #5 :-)
Attachment #541939 - Attachment is patch: true
Attachment #541939 - Flags: review?(neil) → review+
Pushed to comm-central.
http://hg.mozilla.org/comm-central/rev/0ff1183779b1
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: