DecayingSample takes time_point in ctor and uses time_point

Address JIRA RIPD-395

Switches a number of places in Resource::Logic to use abstract_clock::now()
which returns a time_point.  Unfortunately Resource::Logic tracks time
locally also, but with ints, not time_point.  So Resource::Logic uses a
delicate mix of abstract_clock::now() and abstract_clock::elapsed() with
this commit.  That inconsistency could be addressed in a second commit.
I think we're trying to get away from the arbitrary indentation because it isn't clear it really helps comprehension, but more because in practice it gets eroded in editing...