Closed Bug 1350811 Opened 7 years ago Closed 7 years ago

Replace use of int64_t for microseconds by TimeUnit in DecodedStream

Categories

(Core :: Audio/Video: Playback, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → jwwang
Blocks: 1245019
Priority: -- → P3
Comment on attachment 8851477 [details]
Bug 1350811 - Replace use of int64_t for microseconds by TimeUnit in DecodedStream.

https://reviewboard.mozilla.org/r/123786/#review126236

::: dom/media/mediasink/DecodedStream.h:74
(Diff revision 1)
> +  media::TimeUnit FromMicroseconds(int64_t aTime)
> +  {
> +    return media::TimeUnit::FromMicroseconds(aTime);
> +  }

Why this wrapper? saving typing?
Attachment #8851477 - Flags: review?(kaku) → review+
Comment on attachment 8851477 [details]
Bug 1350811 - Replace use of int64_t for microseconds by TimeUnit in DecodedStream.

https://reviewboard.mozilla.org/r/123786/#review126236

> Why this wrapper? saving typing?

Yes.
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3be5ffca0b5a
Replace use of int64_t for microseconds by TimeUnit in DecodedStream. r=kaku
https://hg.mozilla.org/mozilla-central/rev/3be5ffca0b5a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: