New issue
Advanced search Search tips

Issue 637151 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Use TimeDeltas instead of int64ts in our media mojo interfaces

Project Member Reported by w...@chromium.org, Aug 12 2016

Issue description

See for example our mojo VideoFrame.
 
Owner: tguilbert@chromium.org
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc

commit e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc
Author: tguilbert <tguilbert@chromium.org>
Date: Mon Aug 15 21:12:38 2016

Update media mojom files to use TimeDelta instead of int64

Native support for base::TimeDelta was added at the end of May (see
6ae5d56453c79e8377e255f153ce8fe8cafa4d2d)

This change updates the //media/mojo/interfaces/*.mojom to use
mojo.common.mojom.TimeDelta instead of int64, and updates the
implementations of those interfaces.

BUG= 637151 

Review-Url: https://codereview.chromium.org/2240213003
Cr-Commit-Position: refs/heads/master@{#412050}

[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/clients/mojo_renderer.cc
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/clients/mojo_renderer.h
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/common/media_type_converters.cc
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/interfaces/BUILD.gn
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/interfaces/media_types.mojom
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/interfaces/renderer.mojom
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/services/media_mojo_unittest.cc
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/services/mojo_renderer_service.cc
[modify] https://crrev.com/e79bbf93e15d8fd6a16b19ea7e9a54994204f9cc/media/mojo/services/mojo_renderer_service.h

Status: Fixed (was: Started)

Sign in to add a comment