Closed Bug 1163260 Opened 10 years ago Closed 10 years ago

Intermittent browser_page_title.js | correct title - Got initial title, expected new title

Categories

(Firefox :: Session Restore, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 41
Tracking Status
firefox39 --- unaffected
firefox40 --- fixed
firefox41 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- unaffected

People

(Reporter: KWierso, Assigned: ttaubert)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

12:09:48 INFO - 1332 INFO TEST-START | browser/components/sessionstore/test/browser_pageStyle.js 12:09:53 INFO - MEMORY STAT vsize after test: 1167855616 12:09:53 INFO - MEMORY STAT residentFast after test: 300830720 12:09:53 INFO - MEMORY STAT heapAllocated after test: 113136792 12:09:53 INFO - 1333 INFO TEST-OK | browser/components/sessionstore/test/browser_pageStyle.js | took 4546ms 12:09:53 INFO - 1334 INFO TEST-START | browser/components/sessionstore/test/browser_page_title.js 12:09:54 INFO - 1335 INFO checking window state 12:09:54 INFO - 1336 INFO Entering test 12:09:54 INFO - 1337 INFO Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol." {file: "data:text/html,<title>initial%20title</title>" line: 0}] 12:09:54 INFO - 1338 INFO TEST-PASS | browser/components/sessionstore/test/browser_page_title.js | correct title 12:09:54 INFO - 1339 INFO Leaving test 12:09:54 INFO - 1340 INFO Entering test 12:09:54 INFO - 1341 INFO Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol." {file: "data:text/html,<title>initial%20title</title>" line: 0}] 12:09:54 INFO - 1342 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_page_title.js | correct title - Got initial title, expected new title 12:09:54 INFO - Stack trace: 12:09:54 INFO - chrome://mochikit/content/browser-test.js:test_is:925 12:09:54 INFO - chrome://mochitests/content/browser/browser/components/sessionstore/test/browser_page_title.js:null:37 12:09:54 INFO - self-hosted:InterpretGeneratorResume:716 12:09:54 INFO - self-hosted:next:624 12:09:54 INFO - Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:867:23 12:09:54 INFO - this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:746:7 12:09:54 INFO - this.PromiseWalker.scheduleWalkerLoop/<@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:688:37 12:09:54 INFO - Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:688:5 12:09:54 INFO - this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:709:7 12:09:54 INFO - this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:671:7 12:09:54 INFO - TaskImpl_run@resource://gre/modules/Task.jsm:319:13 12:09:54 INFO - Promise*removeTab@resource://testing-common/BrowserTestUtils.jsm:351:1 12:09:54 INFO - promiseRemoveTab@chrome://mochitests/content/browser/browser/components/sessionstore/test/head.js:534:10 12:09:54 INFO - @chrome://mochitests/content/browser/browser/components/sessionstore/test/browser_page_title.js:11:9 12:09:54 INFO - TaskImpl_run@resource://gre/modules/Task.jsm:314:40 12:09:54 INFO - Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:867:23 12:09:54 INFO - this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:746:7 12:09:54 INFO - this.PromiseWalker.scheduleWalkerLoop/<@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:688:37 12:09:54 INFO - Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:688:5 12:09:54 INFO - this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:709:7 12:09:54 INFO - this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:671:7 12:09:54 INFO - onLoad@chrome://mochitests/content/browser/browser/components/sessionstore/test/head.js:291:9 12:09:54 INFO - 1343 INFO Leaving test 12:09:54 INFO - MEMORY STAT vsize after test: 1168117760 12:09:54 INFO - MEMORY STAT residentFast after test: 302530560 12:09:54 INFO - MEMORY STAT heapAllocated after test: 113121016 12:09:54 INFO - 1344 INFO TEST-OK | browser/components/sessionstore/test/browser_page_title.js | took 1065ms 12:09:54 INFO - 1345 INFO TEST-START | browser/components/sessionstore/test/browser_privatetabs.js
I think we need to wait for the event to be dispatched before closing the tab. Otherwise there is no data to flush upon frame script unloading.
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Attachment #8607681 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8607681 [details] [diff] [review] 0001-Bug-1163260-Fix-intermittent-browser_page_title.js-f.patch Review of attachment 8607681 [details] [diff] [review]: ----------------------------------------------------------------- wfm
Attachment #8607681 - Flags: review?(gijskruitbosch+bugs) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: