Auto-formatted all .mojom files.

mojo/services/media/common/interfaces/media_common.mojom

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 [DartPackage="mojo_services"]
 module mojo.media;
 
 // TODO(johngro): When the new extensible MojoResult has been introduced,
 // redefine all of these results to be MojoResults which exist in the Media
 // domain.  For now, comments are included about how the results will probably
 // be bucketed when the time for the transition comes.
 //
 // see
 // 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 won't really cut it either.
maybe the expedient thing for cases like this if we don't want to teach the
formatter about them is to give it clang-format style exclusion comments and
wrap hand-formatted sections like this in those comments

+  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?
 };