Attachment #200166 for bug #351527

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

View | Details | Raw Unified | Return to bug 351527
Collapse All | Expand All

Lines 42-48 Link Here

(-)js/org/eclipse/rwt/widgets/MultiCellWidget.js (-5 / +12 lines)
42	this.initCursor();	42	this.initCursor();
43	this.initTextColor();	43	this.initTextColor();
44	},	44	},
45		45
46	destruct : function() {	46	destruct : function() {
47	this._disposeObjectDeep( "__cellData", 0 );	47	this._disposeObjectDeep( "__cellData", 0 );
48	this._disposeObjectDeep( "__cellNodes", 0 );	48	this._disposeObjectDeep( "__cellNodes", 0 );
Lines 130-135 Link Here
130	},	130	},
131		131
132	members : {	132	members : {
		133
133	// TODO [tb] : clean up api (private/public, order)	134	// TODO [tb] : clean up api (private/public, order)
134		135
135	///////////////////////	136	///////////////////////
Lines 187-196 Link Here
187	},	188	},
188		189
189	setCellWidth : function( cell, width ) {	190	setCellWidth : function( cell, width ) {
190	this._setCellWidth( cell, width );	191	if( this._getCellWidth( cell ) !== width ) {
191	this._invalidateTotalSpacing();	192	this._setCellWidth( cell, width );
192	this._invalidatePreferredInnerWidth();	193	this._invalidateTotalSpacing();
193	this._scheduleLayoutX();	194	this._invalidatePreferredInnerWidth();
		195	this._scheduleLayoutX();
		196	}
194	},	197	},
195		198
196	setCellHeight : function( cell, height ) {	199	setCellHeight : function( cell, height ) {
Lines 384-389 Link Here
384	this.__cellData[ cell ][ 2 ] = width;	387	this.__cellData[ cell ][ 2 ] = width;
385	},	388	},
386		389
		390	_getCellWidth : function( cell ) {
		391	return this.__cellData[ cell ][ 2 ];
		392	},
		393
387	_setCellHeight : function( cell, height ) {	394	_setCellHeight : function( cell, height ) {
388	this.__cellData[ cell ][ 3 ] = height;	395	this.__cellData[ cell ][ 3 ] = height;
389	},	396	},




      assertEquals( 0 , widget.getTotalSpacing() );
      this.disposeWidget( widget );      
    },

    testSettingCellWidthTwice : function() {
      var widget = this.createDefaultWidget();
      this.initWidget( widget, true );
      widget.setCellWidth( 0, 10 );
      assertTrue( widget._isInGlobalJobQueue );
      this.testUtil.flush();
      widget.setCellWidth( 0, 10 );
      assertIdentical( undefined, widget._isInGlobalJobQueue );  
      this.disposeWidget( widget );      
    },

    testInvalidateFrameDimension : function() {
      var widget = this.createDefaultWidget();
      widget.setPadding( 0, 0, 0, 0 );

Return to bug 351527

Lines 286-292 Link Here

(-)js/org/eclipse/rwt/test/tests/MultiCellWidgetTest.js (-1 / +12 lines)
286	assertEquals( 0 , widget.getTotalSpacing() );	286	assertEquals( 0 , widget.getTotalSpacing() );
287	this.disposeWidget( widget );	287	this.disposeWidget( widget );
288	},	288	},
289		289
		290	testSettingCellWidthTwice : function() {
		291	var widget = this.createDefaultWidget();
		292	this.initWidget( widget, true );
		293	widget.setCellWidth( 0, 10 );
		294	assertTrue( widget._isInGlobalJobQueue );
		295	this.testUtil.flush();
		296	widget.setCellWidth( 0, 10 );
		297	assertIdentical( undefined, widget._isInGlobalJobQueue );
		298	this.disposeWidget( widget );
		299	},
		300
290	testInvalidateFrameDimension : function() {	301	testInvalidateFrameDimension : function() {
291	var widget = this.createDefaultWidget();	302	var widget = this.createDefaultWidget();
292	widget.setPadding( 0, 0, 0, 0 );	303	widget.setPadding( 0, 0, 0, 0 );