Closed Bug 761308 Opened 12 years ago Closed 12 years ago

Cleanup CreateElementTxn

Categories

(Core :: DOM: Editor, defect)

Product:
Core
Component:
DOM: Editor
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla16

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Details

Attachments

(2 files)

Part a: Remove unnecessary assertion from Init()
938 bytes, patch
ehsan.akhgari
: review+
Details | Diff | Splinter Review
Part b: DoTransaction()
4.34 KB, patch
ehsan.akhgari
: review+
Details | Diff | Splinter Review 
      No description provided.
Attachment #629913 - Flags: review?(ehsan)

        Attachment #629914 -
        Flags: review?(ehsan)

    
  

        Attachment #629913 -
        Flags: review?(ehsan) → review+

    
    

    
  
Comment on attachment 629914 [details] [diff] [review]
Part b: DoTransaction()

Review of attachment 629914 [details] [diff] [review]:
-----------------------------------------------------------------

(please submit diff -w patches for this kind of stuff in the future!)

        Attachment #629914 -
        Flags: review?(ehsan) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/48ab70bbd706
https://hg.mozilla.org/mozilla-central/rev/9714a877be16
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16

          You need to log in
          before you can comment on or make changes to this bug.