Closed Bug 1764771 Opened 2 years ago Closed 2 years ago

Disable IDBMutableFile by default

Categories

(Core :: Storage: IndexedDB, task)

task

Tracking

()

RESOLVED FIXED
102 Branch
Tracking Status
firefox103 --- fixed

People

(Reporter: saschanaz, Assigned: saschanaz)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

No description provided.

Ok, let's flip the pref.

Pushed by krosylight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/88f605efb204
Disable IDBMutableHandle support by default r=janv,smaug
Backout by mlaza@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d0687914f7bd
Backed out changeset 88f605efb204 for causing xpcshell failures on test_mutableFileUpgrade.js. CLOSED TREE

Backed out changeset 88f605efb204 (Bug 1764771) for causing xpcshell failures on test_mutableFileUpgrade.js.
Backout link
Push with failures <--> X5
Failure Log

Flags: needinfo?(krosylight)
Flags: needinfo?(krosylight)
Backout by nbeleuzu@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/f8470456e36d
Backed out changeset 88f605efb204 for causing xpcshell failures on test_mutableFileUpgrade.js.
Pushed by krosylight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0d9e24627e48
Disable IDBMutableHandle support by default r=janv,smaug
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch

FYI FF102 docs work for this can be tracked in https://github.com/mdn/content/issues/16151#issuecomment-1132419424 (mostly complete)

Somehow the merge commit https://hg.mozilla.org/mozilla-central/rev/58a6343ab33d9ca296fe30e66757301d1cda705d immediately rolled the change back, so FF102 never really had this. Reverting the keyword as such.

NI'ing :aryx for comment #12. I think a merge commit should not roll anything back by itself, right?

Flags: needinfo?(aryx.bugmail)
Pushed by krosylight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d5932b36ee72
Disable IDBMutableHandle support by default r=janv,smaug

(In reply to Kagami :saschanaz from comment #12)

Somehow the merge commit https://hg.mozilla.org/mozilla-central/rev/58a6343ab33d9ca296fe30e66757301d1cda705d immediately rolled the change back, so FF102 never really had this. Reverting the keyword as such.

Hi Kagami, so what happened here was the patch you landed in comment 4 was backed out for xpcshell failures in comment 5/6, that backout was grafted to central in order have a suitable and failure free merge candidate. Grafting that backout created a new central revision for it which was posted by Pulsebot in comment 7. The merge didn't revert anything that wasn't already backed out on autoland.
The next round of merges included your disable patch from comment 8 and was closed as fixed in comment 9.
Treeherder links of those merges are here.

Flags: needinfo?(aryx.bugmail)

Hi Cosmin,

The next round of merges included your disable patch from comment 8 and was closed as fixed in comment 9.

No it didn't, see what actually happened in e.g. FileHandle.webidl (and any other file in this patch): https://hg.mozilla.org/mozilla-central/log/tip/dom/webidl/IDBFileHandle.webidl

Per the log, the merge commit 58a6343ab33d9ca296fe30e66757301d1cda705d did backout the change from the second push in comment #8. That's why I had to push the same patch again in comment #14 even though there was no explicit backout after comment #9.

What's more weird is that the merge commit did not backout dom/indexedDB/test/unit/xpcshell-parent-process.ini. That's the only change that have actually reached the central branch.

Flags: needinfo?(csabou)
Flags: needinfo?(csabou) → needinfo?(aryx.bugmail)

What happened:

  • merge of first landing + grafted backout (= new revision hash to autoland)
  • relanding on autoland
  • mozilla-central gets merged back to autoland and mercurial automatically applies the backout again

Sorry about that.

Mitigation: Directly merge back to autoland after the merge to mozilla-central but this still leaves the risk if the relanding got pushed before the merge from autoland to mozilla-central.

Without grafting, merges would sometimes not possible (e.g. subsequent Nightlies without changes).

Flags: needinfo?(aryx.bugmail)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: