714256 - Remove nsTransactionManager::Lock()/Unlock()

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Comment 1

[(no longer active)]

No editor consumer accesses this object from multiple threads.  I'm not sure about places though...

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

The object is cycle collectable, so it would crash immediately if used in non-main-thread.

Comment 3

[Marco Bonardo [:mak]]

(In reply to Ehsan Akhgari [:ehsan] from comment #1)
> No editor consumer accesses this object from multiple threads.  I'm not sure
> about places though...

Places only uses the transaction manager from js in the ui thread.

Comment 4

[(no longer active)]

Attachment: Patch (v1)

Comment 5

[Mozilla RelEng Bot]

Try run for 6355fe761f05 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=6355fe761f05
Results (out of 206 total builds):
    exception: 1
    success: 183
    warnings: 22
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-6355fe761f05
 Timed out after 06 hours without completing.

Comment 6

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a4a07b81ad67

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/a4a07b81ad67