You are creating a new instance of DrawCircle every time
doIt is called.
The old instance is still there. So you need to either remove the old
view from f1, or get the exact same instance of DrawCircle you created
before, set the new color, and then call invalidate() on the DrawCircle to
force the redraw.