Currently G1ConcurrentMark does flag sanity checking in its constructor; this causes some awkward code with some flag reporting back success.

It would be useful to move this flag sanity checks into g1Arguments.cpp to keep other code clean from that.

Then the flag and flag handling code mentioned above could be removed too.