Avoid unnecessary allocations in AttributeDefinition model resolution

The AttributeDefinition resolveModelAttribute and resolveValue variants that take an OperationContext param are doing pointless allocations to create an anonymous ExpressionResolver inner class that calls the context, which they pass into the method variant that takes an ExpressionResolver. This seems unnecessary; the passed in OperationContext is itself an ExpressionResolver and could be passed along directly.