Closed Bug 369236 Opened 18 years ago Closed 17 years ago

Move IsBidiFormControl from nsHTMLReflowState to nsBlockFrame

Categories

(Core :: Layout: Text and Fonts, defect)

Product:
Core
Component:
Layout: Text and Fonts
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: smontagu, Assigned: smontagu)

References

Details

Attachments

(1 file)

Patch
11.46 KB, patch
roc
: review+
roc
: superreview+
Details | Diff | Splinter Review 
Bug 365130 comment 10:

Robert O'Callahan   2007-01-31 18:30:42 PST

Maybe you should separate out the IsBidiFormControl refactoring so we can land
that right away.
Attached patch Patch — — Splinter Review 

        Attachment #253923 -
        Flags: superreview?(roc)

        Attachment #253923 -
        Flags: review?(roc)

        Attachment #253923 -
        Flags: superreview?(roc)

        Attachment #253923 -
        Flags: superreview+

        Attachment #253923 -
        Flags: review?(roc)

        Attachment #253923 -
        Flags: review+

    
    

    
  
Checked in.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED

    
    

    
  
I'm assuming this is purely code cleanup.  It'd still be nice to have bidi testcases (I don't think we have any), but there's no reason they need to be attached to this bug.
Flags: in-testsuite-

    
  
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: layout.bidi → layout.fonts-and-text

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: