Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Fix a data race flagged by helgrind which swamps any helgrind log -- the actual race is itself is harmless: it could lead to rare spurious double work, but without observable effect other than spending some cycles. The fix does imply an additional atomic bool, which may have a bigger performance impact, but is presumably small enough to also not have an observable difference...