Closed Bug 1163260 Opened 6 years ago Closed 5 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+
https://hg.mozilla.org/mozilla-central/rev/bd6c3d585f23
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
You need to log in before you can comment on or make changes to this bug.