          */
         public function set caretColumnIndex(value:int):void
         {
+            // TBD(hmuller): this short-circuit doens't seem right.  Don't update the _caretColumnIndex
+            // if selectionMode == ROW,ROWS?  What if the selectionMode has changed at commitProperties time?
             if (caretColumnIndex == value || 
                 selectionMode == GridSelectionMode.SINGLE_ROW || 
                 selectionMode == GridSelectionMode.MULTIPLE_ROWS)

            return gridLayout.getItemRendererAt(rowIndex, columnIndex);
        }
        
        
        //--------------------------------------------------------------------------
        //
        //  Method Overrides

            // setters because the gridSelection and gridDimensions might not 
            // be set yet, depending on the order they are initialized when the 
            // grid skin part is added to the data grid.
            
            if (dataProviderChanged || columnsChanged)
            {
                // Remove the current selection and, if requireSelection, make
