PPAPI: Run clang_format.py on content/renderer/pepper

PPAPI: Run clang_format.py on content/renderer/pepper

BUG=345417
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=263356

[bbudge, 2014-04-07 17:02:16]

There are a few places where the formatter is worse, but overall a solid
improvement in readability. It will be great to not have to worry about
indentation and style issues. Enthusiastic LGTM.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/even...
File content/renderer/pepper/event_conversion.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/even...
content/renderer/pepper/event_conversion.cc:464: #define VK_F1 0x70
Aligning defines is now impossible - I'm OK with that.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/even...
content/renderer/pepper/event_conversion.cc:648: // Windows key down events
should always be "raw" to avoid an ASSERT.
Surprising that it "knows" this pertains to the #if. Cool!

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/host...
File content/renderer/pepper/host_array_buffer_var.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/host...
content/renderer/pepper/host_array_buffer_var.cc:64: .release());
I wonder why this didn't go on the previous line?

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/host...
File content/renderer/pepper/host_array_buffer_var.h (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/host...
content/renderer/pepper/host_array_buffer_var.h:31: OVERRIDE;
This is a little weird but consistent I guess.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/host...
File content/renderer/pepper/host_var_tracker_unittest.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/host...
content/renderer/pepper/host_var_tracker_unittest.cc:35: NULL,                  
 NULL, NULL,                    NULL, NULL, NULL, };
Kind of strange formatting.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
File content/renderer/pepper/pepper_plugin_instance_impl.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_plugin_instance_impl.cc:776: // run Stage3D
content on machines that have it blacklisted.
Comment aligned to #if here, probably should be moved after #if.
I'm not suggesting any changes to the CL however.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_plugin_instance_impl.cc:2832: scoped_ptr<const
char * []> argv_array(StringVectorToArgArray(argv_));
This is kind of weird.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
File content/renderer/pepper/pepper_truetype_font_host.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_truetype_font_host.cc:45: OnHostMsgGetTable)
I guess it doesn't look into macros to format.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
File content/renderer/pepper/pepper_video_destination_host.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_video_destination_host.cc:55: NULL /* factory */,
NULL /* registry */, gurl.spec(), &frame_writer))
This is a little less readable.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/plug...
File content/renderer/pepper/plugin_module.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/plug...
content/renderer/pepper/plugin_module.cc:287:
&SetMinimumArrayBufferSizeForShmem};
Hmmm.

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/ppb_...
File content/renderer/pepper/ppb_var_deprecated_impl.cc (right):

https://codereview.chromium.org/225903006/diff/1/content/renderer/pepper/ppb_...
content/renderer/pepper/ppb_var_deprecated_impl.cc:420:
&CreateObjectWithModuleDeprecated, };
I guess initializers don't end with the closing brace on its own line.

[dmichael (off chromium), 2014-04-11 18:20:46]

The CQ bit was checked by dmichael@chromium.org

[commit-bot: I haz the power, 2014-04-11 18:21:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmichael@chromium.org/225903006/20001

[commit-bot: I haz the power, 2014-04-11 21:12:40]

Change committed as 263356