Make code for the AutomaticProfileResetter more readabile (readability review)

chrome/browser/profile_resetter/automatic_profile_resetter_delegate.h

 // Copyright 2013 The Chromium Authors. All rights reserved.

[laefer, 2014/05/25 21:21:58]

https://engdoc.corp.google.com/eng/doc/devguide/cpp/styleguide.shtml?cl=head#...

Please add a comment.

[engedy, 2014/05/26 18:00:46]

Done.

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_PROFILE_RESETTER_AUTOMATIC_PROFILE_RESETTER_DELEGATE_H_
 #define CHROME_BROWSER_PROFILE_RESETTER_AUTOMATIC_PROFILE_RESETTER_DELEGATE_H_
 
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 162 matching lines @@
   void OnBrandcodedDefaultsFetched();
 
   // Called back by the ProfileResetter once resetting the profile settings has
   // been completed. If |old_settings_snapshot| is non-NULL, will compare it to
   // the new settings and send the differences to Google for analysis. Finally,
   // will post |user_callback|.
   void OnProfileSettingsResetCompleted(
       const base::Closure& user_callback,
       scoped_ptr<ResettableSettingsSnapshot> old_settings_snapshot);
 
   Profile* profile_;

[laefer, 2014/05/25 21:21:58]

https://engdoc.corp.google.com/eng/doc/devguide/cpp/styleguide.shtml?cl=head#...

Please add comments for all class member variables. It's okay to use a single
comment for similar variables.

[engedy, 2014/05/26 18:00:46]

Done.

   GlobalErrorService* global_error_service_;
   TemplateURLService* template_url_service_;
 
   scoped_ptr<BrandcodeConfigFetcher> brandcoded_config_fetcher_;
   scoped_ptr<BrandcodedDefaultSettings> brandcoded_defaults_;
 
   const ProfileResetter::ResettableFlags resettable_aspects_;
   scoped_ptr<ProfileResetter> profile_resetter_;
 
   content::NotificationRegistrar registrar_;
@@ skipping 16 matching lines @@
   extensions::OneShotEvent template_url_service_ready_event_;
 
   // This event is signaled once brandcoded default settings have been fetched,
   // or once it has been established that this is not a branded build.
   extensions::OneShotEvent brandcoded_defaults_fetched_event_;
 
   DISALLOW_COPY_AND_ASSIGN(AutomaticProfileResetterDelegateImpl);
 };
 
 #endif  // CHROME_BROWSER_PROFILE_RESETTER_AUTOMATIC_PROFILE_RESETTER_DELEGATE_H_
+