Move URLs out of *.pak files and put them into code.

chrome/browser/views/clear_server_data.cc

Index: chrome/browser/views/clear_server_data.cc
diff --git a/chrome/browser/views/clear_server_data.cc b/chrome/browser/views/clear_server_data.cc
index d01daf6e14c666ca115e27b0c645800e27d86840..2cd3be765471fd5b5432a6e6f5e2acd24047985a 100644
--- a/chrome/browser/views/clear_server_data.cc
+++ b/chrome/browser/views/clear_server_data.cc
@@ -250,9 +250,7 @@ void ClearServerDataView::LinkActivated(views::Link* source,
                      GURL(), NEW_FOREGROUND_TAB, PageTransition::LINK);
     browser->window()->Show();
   } else {
-    browser->OpenURL(GURL(l10n_util::GetStringUTF8(IDS_PRIVACY_DASHBOARD_URL)),
-                     GURL(), NEW_FOREGROUND_TAB, PageTransition::LINK);
-    browser->window()->Show();
+    browser->OpenPrivacyDashboardTabAndActivate();

[tfarina, 2010/11/02 02:30:05]

Removed the call to Show. OK to commit?

[tony, 2010/11/02 18:40:28]

I can't even seem to find where this view is shown.  Seems fine w/o it.

It would be great if you could investigate whether we should be using Activate()
for the flash storage link too.  Maybe it should also be moved into a function
in browser.h/cc (or a more generic method that takes a URL to open and
activate).

   }
 }