[Mac] UI for clearing sync data on the server....

chrome/browser/views/clear_server_data.cc

 // Copyright (c) 2010 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 "chrome/browser/views/clear_server_data.h"
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/command_line.h"
 #include "base/string16.h"
@@ skipping 258 matching lines @@
   // sync disabled)
   UpdateControlEnabledState();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ClearServerDataView, private:
 
 void ClearServerDataView::UpdateControlEnabledState() {
   bool delete_in_progress = false;
 
-  // We only want to call Suceeded/FailedClearingServerData once, not every
-  // time the view is refreshed.  As such, on success/failure we handle that
-  // state and immediately reset things back to CLEAR_NOT_STARTED.
+  // Succeeded/FailedClearingServerData should only be called once, not every
+  // time the view is refreshed.  As such, on success/failure handle that state
+  // and immediately reset things back to CLEAR_NOT_STARTED.
   ProfileSyncService::ClearServerDataState clear_state =
-    profile_->GetProfileSyncService()->GetClearServerDataState();
+      profile_->GetProfileSyncService()->GetClearServerDataState();
   profile_->GetProfileSyncService()->ResetClearServerDataState();
 
   switch (clear_state) {
     case ProfileSyncService::CLEAR_NOT_STARTED:
-      // We can get here when we first start and after a failed clear (which
-      // does not close the tab), do nothing.
+      // This can occur on a first start and after a failed clear (which does

[Raz Mathias, 2010/10/07 20:16:01]

Thanks for fixing spacing here!

+      // not close the tab). Do nothing.
       break;
     case ProfileSyncService::CLEAR_CLEARING:
-        // Clearing buttons on all tabs are disabled at this
-        // point, throbber is going
-        status_label_->SetText(l10n_util::GetString(IDS_CLEAR_DATA_SENDING));
-        status_label_->SetVisible(true);
-        delete_in_progress = true;
+      // Clearing buttons on all tabs are disabled at this point, throbber is
+      // going.
+      status_label_->SetText(l10n_util::GetString(IDS_CLEAR_DATA_SENDING));
+      status_label_->SetVisible(true);
+      delete_in_progress = true;
       break;
     case ProfileSyncService::CLEAR_FAILED:
-        // Show an error and reallow clearing
-        clear_data_parent_window_->FailedClearingServerData();
-        status_label_->SetText(l10n_util::GetString(IDS_CLEAR_DATA_ERROR));
-        status_label_->SetVisible(true);
-        delete_in_progress = false;
+      // Show an error and reallow clearing.
+      clear_data_parent_window_->FailedClearingServerData();
+      status_label_->SetText(l10n_util::GetString(IDS_CLEAR_DATA_ERROR));
+      status_label_->SetVisible(true);
+      delete_in_progress = false;
       break;
     case ProfileSyncService::CLEAR_SUCCEEDED:
-        // Close the dialog box, success!
-        status_label_->SetVisible(false);
-        delete_in_progress = false;
-        clear_data_parent_window_->SucceededClearingServerData();
+      // Close the dialog box, success!
+      status_label_->SetVisible(false);
+      delete_in_progress = false;
+      clear_data_parent_window_->SucceededClearingServerData();
       break;
-    }
+  }
 
   // allow_clear can be false when a local browsing data clear is happening
   // from the neighboring tab.  delete_in_progress means that a clear is
   // pending in the current tab.
   this->clear_server_data_button_->SetEnabled(
       profile_->GetProfileSyncService()->HasSyncSetupCompleted() &&
       !delete_in_progress && allow_clear_);
 
   throbber_->SetVisible(delete_in_progress);
   if (delete_in_progress)
     throbber_->Start();
   else
     throbber_->Stop();
 }