Pre-fill the 'Add Page' bookmark window with the current page (gtk and windows).

chrome/browser/ui/gtk/bookmarks/bookmark_editor_gtk.cc

Index: chrome/browser/ui/gtk/bookmarks/bookmark_editor_gtk.cc
diff --git a/chrome/browser/ui/gtk/bookmarks/bookmark_editor_gtk.cc b/chrome/browser/ui/gtk/bookmarks/bookmark_editor_gtk.cc
index 9e0b4e75a8d0ab683e26e10ec60557cb3f346d9d..4ba1443a9eb5172e0991fccd22438d983cc2205e 100644
--- a/chrome/browser/ui/gtk/bookmarks/bookmark_editor_gtk.cc
+++ b/chrome/browser/ui/gtk/bookmarks/bookmark_editor_gtk.cc
@@ -15,10 +15,13 @@
 #include "chrome/browser/history/history.h"
 #include "chrome/browser/net/url_fixer_upper.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/gtk/bookmarks/bookmark_tree_model.h"
 #include "chrome/browser/ui/gtk/bookmarks/bookmark_utils_gtk.h"
 #include "chrome/browser/ui/gtk/gtk_theme_service.h"
 #include "chrome/browser/ui/gtk/gtk_util.h"
+#include "content/browser/tab_contents/tab_contents.h"
 #include "googleurl/src/gurl.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
@@ -276,10 +279,20 @@ void BookmarkEditorGtk::Init(GtkWindow* parent_window) {
 
   name_entry_ = gtk_entry_new();
   std::string title;
+  std::string url_spec;
   if (details_.type == EditDetails::EXISTING_NODE) {
     title = UTF16ToUTF8(details_.existing_node->GetTitle());
+    url_spec = details_.existing_node->GetURL().spec();
   } else if (details_.type == EditDetails::NEW_FOLDER) {
     title = l10n_util::GetStringUTF8(IDS_BOOMARK_EDITOR_NEW_FOLDER_NAME);
+  } else if (details_.type == EditDetails::NEW_URL) {
+    Browser* browser = BrowserList::GetLastActiveWithProfile(profile_);

[sky, 2011/07/07 16:02:50]

Move this off into bookmark_utils so you don't need to duplicate the logic.
Maybe GetURLAndTitleToBookmarkFromCurrentTab or something. And make use of
GetURLAndTitleToBookmark.

[sadrul, 2011/07/07 16:37:38]

Nice! I wasn't aware of bookmark_utils. Done. Thanks for the suggestion!

+    TabContents* tab_contents = browser ? browser->GetSelectedTabContents()
+                                        : NULL;
+    if (tab_contents) {
+      title = UTF16ToUTF8(tab_contents->GetTitle());
+      url_spec = tab_contents->GetURL().spec();
+    }
   }
   gtk_entry_set_text(GTK_ENTRY(name_entry_), title.c_str());
   g_signal_connect(name_entry_, "changed",
@@ -289,9 +302,6 @@ void BookmarkEditorGtk::Init(GtkWindow* parent_window) {
   GtkWidget* table;
   if (details_.type != EditDetails::NEW_FOLDER) {
     url_entry_ = gtk_entry_new();
-    std::string url_spec;
-    if (details_.type == EditDetails::EXISTING_NODE)
-      url_spec = details_.existing_node->GetURL().spec();
     gtk_entry_set_text(GTK_ENTRY(url_entry_), url_spec.c_str());
     g_signal_connect(url_entry_, "changed",
                      G_CALLBACK(OnEntryChangedThunk), this);