1564894 - favicon-related browser_startup_mainthreadio.js failures with the 201907081345 SQLite snapshot build

Status: RESOLVED FIXED

(Toolkit :: Places, defect, P1)

Description

[Ryan VanderMeulen [:RyanVM]]

When testing the latest SQLite snapshot build on Try (201907081345), I encountered a reproducible failure in browser_startup_mainthreadio.js on Windows 7.

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=255617946&repo=try&lineNumber=2115

TEST-UNEXPECTED-FAIL | browser/base/content/test/performance/browser_startup_mainthreadio.js | unexpected stat on C:\Users\task_1562716606\AppData\Local\Temp\mz_etilqs_nvfkWm1nHVXEYx9 before becoming idle -
Stack trace:
Connection::executeSql CREATE INDEX IF NOT EXISTS moz_icons_iconurlhashindex ON moz_icons (fixed_icon_url_hash)
get (resource://gre/modules/PlacesUtils.jsm:1904:0)
resource://gre/modules/PlacesUtils.jsm:2021:0
get (resource://gre/modules/XPCOMUtils.jsm:124:0)
promiseDBConnection (resource://gre/modules/PlacesUtils.jsm:1495:0)
fetchAnnotatedPages (resource://gre/modules/History.jsm:200:0)
bound (self-hosted:974:0)
ensureInitialized/this._initializePromise< (resource://gre/modules/DownloadHistory.jsm:193:0)
ensureInitialized (resource://gre/modules/DownloadHistory.jsm:189:0)
getList (resource://gre/modules/DownloadHistory.jsm:71:0)
DownloadsDataCtor/this._promiseList< (resource:///modules/DownloadsCommon.jsm:713:0)
promise callback
XREMain::XRE_main

TEST-UNEXPECTED-FAIL | browser/base/content/test/performance/browser_startup_mainthreadio.js | unexpected write on C:\Users\task_1562716606\AppData\Local\Temp\mz_etilqs_nvfkWm1nHVXEYx9 before becoming idle -
Stack trace:
Connection::executeSql CREATE INDEX IF NOT EXISTS moz_icons_iconurlhashindex ON moz_icons (fixed_icon_url_hash)
get (resource://gre/modules/PlacesUtils.jsm:1904:0)
resource://gre/modules/PlacesUtils.jsm:2021:0
get (resource://gre/modules/XPCOMUtils.jsm:124:0)
promiseDBConnection (resource://gre/modules/PlacesUtils.jsm:1495:0)
fetchAnnotatedPages (resource://gre/modules/History.jsm:200:0)
bound (self-hosted:974:0)
ensureInitialized/this._initializePromise< (resource://gre/modules/DownloadHistory.jsm:193:0)
ensureInitialized (resource://gre/modules/DownloadHistory.jsm:189:0)
getList (resource://gre/modules/DownloadHistory.jsm:71:0)
DownloadsDataCtor/this._promiseList< (resource:///modules/DownloadsCommon.jsm:713:0)
promise callback
XREMain::XRE_main

TEST-UNEXPECTED-FAIL | browser/base/content/test/performance/browser_startup_mainthreadio.js | unexpected write on C:\Users\task_1562716606\AppData\Local\Temp\mz_etilqs_nvfkWm1nHVXEYx9 before becoming idle -
Stack trace:
Connection::executeSql CREATE INDEX IF NOT EXISTS moz_pages_w_icons_urlhashindex ON moz_pages_w_icons (page_url_hash)
get (resource://gre/modules/PlacesUtils.jsm:1904:0)
resource://gre/modules/PlacesUtils.jsm:2021:0
get (resource://gre/modules/XPCOMUtils.jsm:124:0)
promiseDBConnection (resource://gre/modules/PlacesUtils.jsm:1495:0)
fetchAnnotatedPages (resource://gre/modules/History.jsm:200:0)
bound (self-hosted:974:0)
ensureInitialized/this._initializePromise< (resource://gre/modules/DownloadHistory.jsm:193:0)
ensureInitialized (resource://gre/modules/DownloadHistory.jsm:189:0)
getList (resource://gre/modules/DownloadHistory.jsm:71:0)
DownloadsDataCtor/this._promiseList< (resource:///modules/DownloadsCommon.jsm:713:0)
promise callback
XREMain::XRE_main

Comment 1

[Marco Bonardo [:mak] (OoO 09-23 Aug)]

On 32 bit we don't enable TEMP_STORE by default, that means databases may end up creating tmp files when necessary.
https://searchfox.org/mozilla-central/rev/40ef22080910c2e2c27d9e2120642376b1d8b8b2/third_party/sqlite3/src/moz.build#90

The favicons creation connection is a short lived one, so we can use temp_store for it.

Comment 2

[Marco Bonardo [:mak] (OoO 09-23 Aug)]

Attachment: Bug 1564894 - Use a memory temp store when creating the favicons database. r=Standard8

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/723cf15f8cb4
Use a memory temp store when creating the favicons database. r=Standard8

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/723cf15f8cb4