[OpenLayers-Trac] Re: [OpenLayers] #2850: Control should set this.div to null in destroy

OpenLayers trac-20090302 at openlayers.org
Fri Sep 24 11:03:22 EDT 2010


#2850: Control should set this.div to null in destroy
----------------------+-----------------------------------------------------
  Reporter:  bartvde  |       Owner:              
      Type:  bug      |      Status:  reopened    
  Priority:  minor    |   Milestone:  2.11 Release
 Component:  Control  |     Version:  2.10        
Resolution:           |    Keywords:              
     State:  Review   |  
----------------------+-----------------------------------------------------

Comment(by jorix):

 Replying to [comment:9 bartvde]:
 > All control tests pass again in FF 3.6 and IE6, please review.
 >
 > This should teach me to think an innocent one-line patch exists ... :-)
 Is best left three test in panel.html. Only need to change line 246 in
 panel.html removing "panel." leaving:
 {{{
 t.ok (div.innerHTML == ""
 }}}
 and test works.[[BR]]
 I neglected to remove the "panel." in that line in my ticket #2835, sorry.

-- 
Ticket URL: <http://trac.openlayers.org/ticket/2850#comment:10>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list