Update sessionrestore module to not use remote pages and correctly handles dtds

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

unspecified

Firefox Tracking Flags

(firefox16 fixed, firefox17 fixed, firefox18 fixed, firefox19 fixed, firefox-esr10 fixed)

Details

(Whiteboard: [lib])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
The sessionrestore module is a bit broken when it comes to the dtds. So any action to undo a tab/window close will fail. Further its API test uses a remote page which has to be local. Those changes make it easier for us for PHX.
(Assignee)

Comment 1

5 years ago
Created attachment 675414 [details] [diff] [review]
Patch v1
Attachment #675414 - Flags: review?(dave.hunt)
(Assignee)

Updated

5 years ago
Blocks: 805705
Attachment #675414 - Flags: review?(dave.hunt) → review+
(Assignee)

Comment 2

5 years ago
http://hg.mozilla.org/qa/mozmill-tests/rev/ba43e72bb8e3 (default)
status-firefox-esr10: --- → affected
status-firefox16: --- → affected
status-firefox17: --- → affected
status-firefox18: --- → affected
status-firefox19: --- → fixed
(Assignee)

Comment 3

5 years ago
All tests are passing, so lets get the patch backported.

http://hg.mozilla.org/qa/mozmill-tests/rev/1c140308b898 (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/02ef81a0b8b9 (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/6cde8d260517 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/d0f9168c087b (esr10)
Status: NEW → RESOLVED
Last Resolved: 5 years ago
status-firefox-esr10: affected → fixed
status-firefox16: affected → fixed
status-firefox17: affected → fixed
status-firefox18: affected → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.