RefCounted types should not have public destructors, Linux fixes

chrome/browser/ui/login/login_prompt_gtk.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 "chrome/browser/ui/login/login_prompt.h"
 
 #include <gtk/gtk.h>
 
 #include "base/string16.h"
 #include "base/utf_string_conversions.h"
@@ skipping 27 matching lines @@
 class LoginHandlerGtk : public LoginHandler,
                         public ConstrainedWindowGtkDelegate {
  public:
   LoginHandlerGtk(net::AuthChallengeInfo* auth_info, net::URLRequest* request)
       : LoginHandler(auth_info, request),
         username_entry_(NULL),
         password_entry_(NULL),
         ok_(NULL) {
   }
 
-  virtual ~LoginHandlerGtk() {
-    root_.Destroy();
-  }
-
   // LoginModelObserver implementation.
   virtual void OnAutofillDataAvailable(const string16& username,
                                        const string16& password) {
     DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
     // NOTE: Would be nice to use gtk_entry_get_text_length, but it is fairly
     // new and not always in our GTK version.
     if (strlen(gtk_entry_get_text(GTK_ENTRY(username_entry_))) == 0) {
       gtk_entry_set_text(GTK_ENTRY(username_entry_),
                          UTF16ToUTF8(username).c_str());
@@ skipping 73 matching lines @@
   virtual void DeleteDelegate() {
     DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
     // The constrained window is going to delete itself; clear our pointer.
     SetDialog(NULL);
     SetModel(NULL);
 
     ReleaseSoon();
   }
 
+ protected:
+  virtual ~LoginHandlerGtk() {
+    root_.Destroy();
+  }
+
  private:
   friend class LoginPrompt;
 
   CHROMEGTK_CALLBACK_0(LoginHandlerGtk, void, OnOKClicked);
   CHROMEGTK_CALLBACK_0(LoginHandlerGtk, void, OnCancelClicked);
   CHROMEGTK_CALLBACK_1(LoginHandlerGtk, void, OnPromptHierarchyChanged,
                        GtkWidget*);
 
   // The GtkWidgets that form our visual hierarchy:
   // The root container we pass to our parent.
@@ skipping 28 matching lines @@
   // button the default action and mess with the focus.
   gtk_widget_set_can_default(ok_, TRUE);
   gtk_widget_grab_default(ok_);
 }
 
 // static
 LoginHandler* LoginHandler::Create(net::AuthChallengeInfo* auth_info,
                                    net::URLRequest* request) {
   return new LoginHandlerGtk(auth_info, request);
 }