Adding a dialog on click of "add to homescreen" menu item to edit the title.

chrome/browser/android/shortcut_helper.cc

 // Copyright 2013 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/android/shortcut_helper.h"
 
 #include <jni.h>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/basictypes.h"
 #include "base/location.h"
 #include "base/strings/string16.h"
 #include "base/threading/worker_pool.h"
 #include "chrome/browser/android/tab_android.h"
 #include "chrome/browser/favicon/favicon_service.h"
 #include "chrome/browser/favicon/favicon_service_factory.h"
 #include "chrome/common/cancelable_task_tracker.h"
 #include "chrome/common/render_messages.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/common/frame_navigate_params.h"
 #include "jni/ShortcutHelper_jni.h"
 #include "ui/gfx/android/java_bitmap.h"
 #include "ui/gfx/codec/png_codec.h"
 #include "ui/gfx/color_analysis.h"
 #include "url/gurl.h"
 
-ShortcutBuilder::ShortcutBuilder(content::WebContents* web_contents)
+ShortcutBuilder::ShortcutBuilder(content::WebContents* web_contents,
+                                 string16 title)
     : is_webapp_capable_(false) {
   Observe(web_contents);
   url_ = web_contents->GetURL();
-  title_ = web_contents->GetTitle();
+  if (title.length() != 0)

[gone, 2013/09/13 00:31:53]

nit: > 0

[Ramya, 2013/09/13 01:04:34]

Done.

+    title_ = title;
+  else
+    title_ = web_contents->GetTitle();
 
   // Send a message to the renderer to retrieve information about the page.
   Send(new ChromeViewMsg_RetrieveWebappInformation(routing_id(), url_));
 }
 
 void ShortcutBuilder::OnDidRetrieveWebappInformation(bool success,
                                                      bool is_webapp_capable,
                                                      const GURL& expected_url) {
   Profile* profile =
       Profile::FromBrowserContext(web_contents()->GetBrowserContext());
@@ skipping 57 matching lines @@
   Destroy();
 }
 
 void ShortcutBuilder::Destroy() {
   if (cancelable_task_tracker_.HasTrackedTasks()) {
     cancelable_task_tracker_.TryCancelAll();
   }
   delete this;
 }
 
-void ShortcutHelper::AddShortcut(content::WebContents* web_contents) {
+void ShortcutHelper::AddShortcut(content::WebContents* web_contents,
+                                 string16 title) {
   // The ShortcutBuilder deletes itself when it's done.
-  new ShortcutBuilder(web_contents);
+  new ShortcutBuilder(web_contents, title);
 }
 
 bool ShortcutHelper::RegisterShortcutHelper(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 void ShortcutHelper::AddShortcutInBackground(
     const GURL& url,
     const string16& title,
     bool is_webapp_capable,
@@ skipping 36 matching lines @@
                                   r_value,
                                   g_value,
                                   b_value,
                                   is_webapp_capable);
 }
 
 // Adds a shortcut to the current URL to the Android home screen, firing
 // background tasks to pull all the data required.
 // Note that we don't actually care about the tab here -- we just want
 // its otherwise inaccessible WebContents.
-static void AddShortcut(JNIEnv* env, jclass clazz, jint tab_android_ptr) {
+static void AddShortcut(JNIEnv* env,
+                        jclass clazz,
+                        jint tab_android_ptr,
+                        jstring title) {
   TabAndroid* tab = reinterpret_cast<TabAndroid*>(tab_android_ptr);
-  ShortcutHelper::AddShortcut(tab->web_contents());
+  ShortcutHelper::AddShortcut(
+      tab->web_contents(),
+      base::android::ConvertJavaStringToUTF16(env, title));
 }