Make code for the AutomaticProfileResetter more readabile (readability review)

chrome/browser/profile_resetter/automatic_profile_resetter_delegate.cc

 // Copyright 2013 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.
 
+// Contains the implementation for AutomaticProfileResetterDelegateImpl.

[Peter Kasting, 2014/05/28 18:46:32]

Personally, I'm skeptical that this comment adds anything useful to the reader. 
I realize the style guide requests file comments in all files, but I would put a
file comment here only if you have anything to say about the actual
implementation, or if you need to direct the reader to a specific place to find
documentation about the contents of the file.

[engedy, 2014/05/30 14:38:47]

I agree, actually I have added this after quite some hesitation. Now I have
removed it.

+
 #include "chrome/browser/profile_resetter/automatic_profile_resetter_delegate.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/md5.h"
 #include "base/memory/scoped_vector.h"
@@ skipping 373 matching lines @@
     if (difference) {
       old_settings_snapshot->Subtract(new_settings_snapshot);
       std::string report =
           SerializeSettingsReport(*old_settings_snapshot, difference);
       SendFeedback(report);
     }
   }
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE, user_callback);
 }
+

[Peter Kasting, 2014/05/28 18:46:32]

Nit: Don't add a trailing blank line

[engedy, 2014/05/30 14:38:47]

Done.