Closed Bug 1358368 Opened 4 years ago Closed 4 years ago
_root _icons .js | test _remove Pages By Timeframe - [test _remove Pages By Timeframe : 84] There should be no icon entry - 1 == 0
Bug 1358368 - Ensure we read values from a current db snapshot in places/tests/favicons/test_root_icons.js.
59 bytes, text/x-review-board-request
Taking for investigation
Assignee: nobody → mak77
Priority: -- → P2
my only suspect so far is that the concurrent connection is reading from the db while the db is checkpointing the change, that means it reads an outdated snapshot. adding a promiseUpdatePlaces may do in such a case. I cannot reproduce locally.
Comment on attachment 8860343 [details] Bug 1358368 - Ensure we read values from a current db snapshot in places/tests/favicons/test_root_icons.js. https://reviewboard.mozilla.org/r/132386/#review135260 This seems reasonable to try. r=Standard8
Attachment #8860343 - Flags: review?(standard8) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/dea1c646413a Ensure we read values from a current db snapshot in places/tests/favicons/test_root_icons.js. r=standard8
You need to log in before you can comment on or make changes to this bug.