clang-format //shell and add a PRESUBMIT.py to keep it clang-formatted.

clang-format //shell and add a PRESUBMIT.py to keep it clang-formatted.

(One day, once everything is clang-formatted, we'll just add that check
to the root PRESUBMIT.py. Directories like //base remain problematic for
now though.)

R=jamesr@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/32952271830d417e7ae98b08aecc0f0760648a37

[jamesr, 2015-02-11 23:44:48]

lgtm

https://codereview.chromium.org/866283007/diff/1/shell/application_manager/ap...
File shell/application_manager/application_manager_unittest.cc (right):

https://codereview.chromium.org/866283007/diff/1/shell/application_manager/ap...
shell/application_manager/application_manager_unittest.cc:415: virtual void
OnApplicationError(const GURL& url) override {}
while you're here, this shouldn't have both "virtual" and "override" (the clang
checker is just ignoring this because of the test namespace i believe)

[viettrungluu, 2015-02-11 23:58:08]

New patchsets have been uploaded after l-g-t-m from jamesr@chromium.org

[viettrungluu, 2015-02-11 23:59:03]

https://codereview.chromium.org/866283007/diff/1/shell/application_manager/ap...
File shell/application_manager/application_manager_unittest.cc (right):

https://codereview.chromium.org/866283007/diff/1/shell/application_manager/ap...
shell/application_manager/application_manager_unittest.cc:415: virtual void
OnApplicationError(const GURL& url) override {}
On 2015/02/11 23:44:48, jamesr wrote:
> while you're here, this shouldn't have both "virtual" and "override" (the
clang
> checker is just ignoring this because of the test namespace i believe)

Fixed. I wonder where else we make that mistake.

[viettrungluu, 2015-02-12 00:32:07]

Committed patchset #2 (id:20001) manually as
32952271830d417e7ae98b08aecc0f0760648a37 (presubmit successful).