GitLab

Before using the UsageTimeLimit on the TimeLimitProcessor we convert this to higher level structs, that allow the processor to work better. I was going to submit all together, but it was too big, so I split into two cls.

These structs here didn't need to have their interfaces public, since its going to be used solely by the TimeLimitProcessor. However, I exposed its interface on the internal namespace in order to make it testable. Usually, we don't want to test private code, but these converters have a complex logic, and I thought it was better to have this explicitly tested.

Bug: 823536
Change-Id: Ia90c5e83b5d0f149ae93024a1671b93850584f91
Reviewed-on: https://chromium-review.googlesource.com/1071687
Commit-Queue: Rahul Chaturvedi <rkc@chromium.org>
Reviewed-by: Rahul Chaturvedi <rkc@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565036}