Closed
Bug 1394767
Opened 7 years ago
Closed 7 years ago
restoreTab shouldn't use NS_ASSERT
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: dao, Assigned: dao)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
from bug 1388628 comment 10:
> Okay, I can reproduce with a file:// homepage in nightly.
>
> This seems relevant:
>
> A coding exception was thrown in a Promise resolution callback.
> See
> https://developer.mozilla.org/Mozilla/JavaScript_code_modules/Promise.jsm/
> Promise
>
> Full message: TypeError: access to strict mode caller function is censored
> Full stack: NS_ASSERT@resource://gre/modules/debug.js:50:7
> restoreTab@resource:///modules/sessionstore/SessionStore.jsm:3552:5
> restoreTabs@resource:///modules/sessionstore/SessionStore.jsm:3539:7
> ssi_restoreWindow@resource:///modules/sessionstore/SessionStore.jsm:3404:7
> ssi_restoreWindows@resource:///modules/sessionstore/SessionStore.jsm:3484:5
> initializeWindow@resource:///modules/sessionstore/SessionStore.jsm:1158:11
> onBeforeBrowserWindowShown/<@resource:///modules/sessionstore/SessionStore.
> jsm:1307:9
> process@resource://gre/modules/Promise-backend.js:922:23
> walkerLoop@resource://gre/modules/Promise-backend.js:806:7
> scheduleWalkerLoop/<@resource://gre/modules/Promise-backend.js:742:11
restoreTab should not break the whole session restore process when this happens. It should instead log an error and return, or something like that.
Updated•7 years ago
|
Blocks: ss-reliability
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8905460 [details]
Bug 1394767 - Log an error and return in unexpected situations rather than calling NS_ASSERT and letting the subsequent code fail.
https://reviewboard.mozilla.org/r/177274/#review182286
Thanks! Can you update the strings passed into `Cu.reportError()` to be proper sentences, starting with capitals and ending with a dot?
Attachment #8905460 -
Flags: review?(mdeboer) → review+
Assignee | ||
Comment 3•7 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #1)
> Returning in onBrowserCrashed if the browser isn't remote breaks various
> tests, so I simply removed this seemingly bogus check.
Try run with the check and return:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2baea9efe2294df9660e32d5a5addcb982eb033c
Without that check:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9d9f1cfea23b0f9aac024fff68dfa8ee6b3f4dd9
Comment hidden (mozreview-request) |
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b4b9eaba235e
Log an error and return in unexpected situations rather than calling NS_ASSERT and letting the subsequent code fail. r=mikedeboer
Comment 6•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in
before you can comment on or make changes to this bug.
Description
•