Linux/ChromeOS Chromium style checker cleanup, chrome/browser edition.

chrome/browser/autofill/autocomplete_history_manager_unittest.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.
 
 #include <vector>
 
 #include "base/memory/ref_counted.h"
 #include "base/string16.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/autocomplete_history_manager.h"
@@ skipping 161 matching lines @@
 
 class AutocompleteHistoryManagerStubSend : public AutocompleteHistoryManager {
  public:
   explicit AutocompleteHistoryManagerStubSend(
       WebContents* web_contents,
       Profile* profile,
       scoped_ptr<AutofillWebDataService> wds)
       : AutocompleteHistoryManager(web_contents, profile, wds.Pass()) {}
 
   // Intentionally swallow the message.
-  virtual bool Send(IPC::Message* message) { delete message; return true; }
+  virtual bool Send(IPC::Message* message) OVERRIDE {
+    delete message;

[Nico, 2013/02/06 22:22:58]

Out of curiosity, did you rewrap manually or is this clang-format doing its
thing?

[Ryan Sleevi, 2013/02/06 22:29:07]

Both, depending.

In the original 751 file CL, there were 57 files that violated the 80 character
line length limits, and 36 that used different indentation (eg: the first and
second arguments to a function were no longer aligned by using virtual).

For all but about 12 of those, I just used "vi $(git diff-index)" and
clang-format to fix (since clang-format-diff is broken atm).

clang-format doesn't handle formatting OVERRIDE/const properly (I need to file a
bug), so those I fixed by hand.

+    return true;
+  }
 };
 
 }  // namespace
 
 // Make sure our external delegate is called at the right time.
 TEST_F(AutocompleteHistoryManagerTest, ExternalDelegate) {
   // Local version with a stubbed out Send()
   AutocompleteHistoryManagerStubSend autocomplete_history_manager(
       web_contents(),
       &profile_,
       scoped_ptr<AutofillWebDataService>(
           new AutofillWebDataServiceImpl(web_data_service_)));
 
   MockAutofillExternalDelegate external_delegate(web_contents());
   EXPECT_CALL(external_delegate, OnSuggestionsReturned(_, _,  _,  _,  _));
   autocomplete_history_manager.SetExternalDelegate(&external_delegate);
 
   // Should trigger a call to OnSuggestionsReturned, verified by the mock.
   autocomplete_history_manager.SendSuggestions(NULL);
 }