PPAPI: Format ppapi/thunk using clang-format.

ppapi/thunk/ppb_url_loader_thunk.cc

 // Copyright (c) 2012 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.
 
-// From ppb_url_loader.idl modified Tue Aug 20 08:13:36 2013.
+// From ppb_url_loader.idl modified Tue May  7 14:43:00 2013.
 
 #include <string.h>
 
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/c/ppb_url_loader.h"
 #include "ppapi/shared_impl/tracked_callback.h"
 #include "ppapi/thunk/enter.h"
 #include "ppapi/thunk/ppapi_thunk_export.h"
 #include "ppapi/thunk/ppb_url_loader_api.h"
@@ skipping 72 matching lines @@
 }
 
 int32_t ReadResponseBody(PP_Resource loader,
                          void* buffer,
                          int32_t bytes_to_read,
                          struct PP_CompletionCallback callback) {
   VLOG(4) << "PPB_URLLoader::ReadResponseBody()";
   EnterResource<PPB_URLLoader_API> enter(loader, callback, true);
   if (enter.failed())
     return enter.retval();
-  return enter.SetResult(enter.object()->ReadResponseBody(buffer,
-                                                          bytes_to_read,
-                                                          enter.callback()));
+  return enter.SetResult(enter.object()->ReadResponseBody(
+      buffer, bytes_to_read, enter.callback()));
 }
 
 int32_t FinishStreamingToFile(PP_Resource loader,
                               struct PP_CompletionCallback callback) {
   VLOG(4) << "PPB_URLLoader::FinishStreamingToFile()";
   EnterResource<PPB_URLLoader_API> enter(loader, callback, true);
   if (enter.failed())
     return enter.retval();
-  return enter.SetResult(enter.object()->FinishStreamingToFile(
-      enter.callback()));
+  return enter.SetResult(
+      enter.object()->FinishStreamingToFile(enter.callback()));
 }
 
 void Close(PP_Resource loader) {
   VLOG(4) << "PPB_URLLoader::Close()";
   EnterResource<PPB_URLLoader_API> enter(loader, true);
   if (enter.failed())
     return;
   enter.object()->Close();
 }
 
-const PPB_URLLoader_1_0 g_ppb_urlloader_thunk_1_0 = {
-  &Create,
-  &IsURLLoader,
-  &Open,
-  &FollowRedirect,
-  &GetUploadProgress,
-  &GetDownloadProgress,
-  &GetResponseInfo,
-  &ReadResponseBody,
-  &FinishStreamingToFile,
-  &Close
+const PPB_URLLoader_1_0 g_ppb_urlloader_thunk_1_0 = {  //
+    &Create,                                           //
+    &IsURLLoader,                                      //
+    &Open,                                             //
+    &FollowRedirect,                                   //
+    &GetUploadProgress,                                //
+    &GetDownloadProgress,                              //
+    &GetResponseInfo,                                  //
+    &ReadResponseBody,                                 //
+    &FinishStreamingToFile,                            //
+    &Close                                             //

[Nico, 2014/04/17 18:37:36]

Eww, that's pretty horrible, no?

Have you tried if this is better with the just-updated clang-format that today's
"gclient sync" will get you?

[dmichael (off chromium), 2014/04/17 19:11:13]

Yes, exactly why I had filed that bug...  thank you for helping me with that!
No, I misunderstood and thought it wasn't going to get resolved the way I had
hoped. Thanks, I'll do that before landing, since the CQ rejected this anyway.

 };
 
 }  // namespace
 
 PPAPI_THUNK_EXPORT const PPB_URLLoader_1_0* GetPPB_URLLoader_1_0_Thunk() {
   return &g_ppb_urlloader_thunk_1_0;
 }
 
 }  // namespace thunk
 }  // namespace ppapi