Closed Bug 1533037 Opened 5 years ago Closed 4 years ago

Make it more clear how you should modify synced fields

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

Status:
RESOLVED WONTFIX
Project Flags:
Fission Milestone Future

People

(Reporter: farre, Assigned: farre)

References

(Blocks 1 open bug)

Details

BrowsingContext ctxt;
{
auto tx = ctxt->Mutate();
tx.SetFoo(42);
}

The intention here is to have BrowsingContext::Mutate return a BrowsingContext::Transaction, which is a RAII class that commits on scope exit, unless all it's values are None, wich could happen if someone calls Commit before scope exit.

Assignee: nobody → afarre
Blocks: 1519151
Status: NEW → ASSIGNED
Priority: -- → P3
Fission Milestone: --- → M2
Component: DOM → DOM: Core & HTML
Blocks: improve-bc
No longer blocks: 1519151
Fission Milestone: M2 → M3
Fission Milestone: M3 → M4
Fission Milestone: M4 → Future

Let's not change how to work with synced fields any more than what's planned for bug 1613431, so closing.

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.