Closed Bug 1159578 Opened 4 years ago Closed 4 years ago
Bug 1138260 introduced the strongly typed media::Microseconds object. The main aim of this type was to avoid common bugs comparing seconds with microseconds and incorrectly use them interchangeably. We should have a TimeUnit object which doesn't represent a particular time unit, but instead would be used to convert to microseconds or seconds or what else. The internal implementation of a TimeUnit would likely use microseconds.
Add TimeUnit object, a strongly typed unit of time
Comment on attachment 8599640 [details] [diff] [review] Add a TimeUnit object will change so it can detect overflow
Use CheckedInt64 instead
Attachment #8600142 - Flags: review?(matt.woodrow)
Attachment #8600142 - Flags: review?(matt.woodrow) → review+
You need to log in before you can comment on or make changes to this bug.