Chromium Code Reviews| Index: mojo/services/media/common/interfaces/media_common.mojom |
| diff --git a/mojo/services/media/common/interfaces/media_common.mojom b/mojo/services/media/common/interfaces/media_common.mojom |
| index e5edc95d8f0cc6d07084a26bb582f7379916fa9a..68cc26fa8955e793f240f4ee27dbc3b35882845f 100644 |
| --- a/mojo/services/media/common/interfaces/media_common.mojom |
| +++ b/mojo/services/media/common/interfaces/media_common.mojom |
| @@ -14,23 +14,23 @@ module mojo.media; |
| // https://docs.google.com/document/d/1A5QL20LzfRDOmsQKyHnA5HK1ZlmrLwAQO9y98GiDNyI/edit |
| // |
| enum MediaResult { |
| - OK = 0, // direct map to MojoResult::OK |
| - UNKNOWN_ERROR = -1, // bucket -> UNKNOWN |
| - INTERNAL_ERROR = -2, // bucket -> INTERNAL |
| - NOT_IMPLEMENTED = -3, // bucket -> UNIMPLEMENTED |
| - UNSUPPORTED_OPERATION = -4, // bucket -> UNIMPLEMENTED, FAILED_PRECON or |
| - // INVALID_ARG, depending on reason |
| - UNSUPPORTED_CONFIG = -5, // bucket -> FAILED_PRECON or INVALID_ARG, or |
| - // RESOURCE_EXHAUSTED |
| - INVALID_ARGUMENT = -6, // bucket -> INVALID_ARGUMENT |
| - INSUFFICIENT_RESOURCES = -7, // bucket -> RESOURCE_EXHAUSTED |
| - BAD_STATE = -8, // bucket -> FAILED_PRECON |
| - BUF_OVERFLOW = -9, // bucket -> RESOURCE_EXHAUSTED |
| - FLUSHED = -10, // bucket -> ABORTED or CANCELLED |
| - BUSY = -11, // bucket -> BUSY |
| - PROTOCOL_ERROR = -12, // bucket -> FAILED_PRECON |
| - ALREADY_EXISTS = -13, // bucket -> ALREADY_EXISTS |
| - NOT_FOUND = -14, // bucket -> NOT_FOUND |
| - SHUTTING_DOWN = -15, // bucket -> FAILED_PRECON |
| - CONNECTION_LOST = -16, // bucket -> INTERNAL? |
| + OK = 0, // direct map to MojoResult::OK |
| + UNKNOWN_ERROR = -1, // bucket -> UNKNOWN |
| + INTERNAL_ERROR = -2, // bucket -> INTERNAL |
| + NOT_IMPLEMENTED = -3, // bucket -> UNIMPLEMENTED |
| + UNSUPPORTED_OPERATION = -4, // bucket -> UNIMPLEMENTED, FAILED_PRECON or |
| + // INVALID_ARG, depending on reason |
|
vtl
2016/02/29 23:45:37
This is a bit of a trainwreck.
jamesr
2016/02/29 23:49:02
yeah, this is awful. aligning just the comments wo
|
| + UNSUPPORTED_CONFIG = -5, // bucket -> FAILED_PRECON or INVALID_ARG, or |
| + // RESOURCE_EXHAUSTED |
| + INVALID_ARGUMENT = -6, // bucket -> INVALID_ARGUMENT |
| + INSUFFICIENT_RESOURCES = -7, // bucket -> RESOURCE_EXHAUSTED |
| + BAD_STATE = -8, // bucket -> FAILED_PRECON |
| + BUF_OVERFLOW = -9, // bucket -> RESOURCE_EXHAUSTED |
| + FLUSHED = -10, // bucket -> ABORTED or CANCELLED |
| + BUSY = -11, // bucket -> BUSY |
| + PROTOCOL_ERROR = -12, // bucket -> FAILED_PRECON |
| + ALREADY_EXISTS = -13, // bucket -> ALREADY_EXISTS |
| + NOT_FOUND = -14, // bucket -> NOT_FOUND |
| + SHUTTING_DOWN = -15, // bucket -> FAILED_PRECON |
| + CONNECTION_LOST = -16, // bucket -> INTERNAL? |
| }; |