I don't think this is correct. It is possible for cancelEditing to be called multiple times. Without the test for pendingDispose, it would attempt to destroy the cell editor multiple times. Did you encounter a problem that caused you to remove this condition?