Added GetFormattedUrlFromOriginalDistillerUrl to DomDistillerTabUtils

chrome/browser/dom_distiller/tab_utils_android.cc

 // Copyright 2014 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/dom_distiller/tab_utils_android.h"
 
 #include <string>
 
+#include "base/android/jni_string.h"
+#include "base/prefs/pref_service.h"
 #include "chrome/browser/dom_distiller/tab_utils.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/common/pref_names.h"
 #include "content/public/browser/web_contents.h"
+#include "content/public/common/content_constants.h"
 #include "jni/DomDistillerTabUtils_jni.h"
+#include "net/base/net_util.h"
+#include "url/gurl.h"
 
 namespace android {
 
 void DistillCurrentPageAndView(JNIEnv* env,
                                jclass clazz,
                                jobject j_web_contents) {
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(j_web_contents);
   ::DistillCurrentPageAndView(web_contents);
 }
 
+jstring GetFormattedUrlFromOriginalDistillerUrl(JNIEnv* env,
+                                                jclass clazz,
+                                                jstring j_url) {
+  GURL url(base::android::ConvertJavaStringToUTF8(env, j_url));
+  Profile* profile = ProfileManager::GetLastUsedProfile();
+  std::string languages;  // Empty if Profile cannot be retrieved.
+  if (profile) {
+    languages = profile->GetPrefs()->GetString(prefs::kAcceptLanguages);
+  }
+
+  // Note that we can't unescape spaces here, because if the user copies this

[nyquist, 2014/08/11 20:35:48]

I believe this is about net::FormatUrl, not the part about length. Could you
move the comment down to where Robert mentioned, immediately preceding the
discussed line? (ie. just above FormatUrl)?

[sunangel, 2014/08/12 01:54:39]

Done.

+  // and pastes it into another program, that program may think the URL ends at
+  // the space.
+  if (url.spec().length() > content::kMaxURLDisplayChars)
+    url = url.IsStandard() ? url.GetOrigin() : GURL(url.scheme() + ":");
+
+  base::string16 formatted_url = net::FormatUrl(url,
+                                                languages,
+                                                net::kFormatUrlOmitAll,
+                                                net::UnescapeRule::NORMAL,
+                                                NULL,
+                                                NULL,
+                                                NULL);
+  base::android::ScopedJavaLocalRef<jstring> j_formatted_url =

[nyquist, 2014/08/11 20:35:48]

Is it possible to just
return base::android::ConverTUT...().Release()
here?

[sunangel, 2014/08/12 01:54:39]

Done.

+      base::android::ConvertUTF16ToJavaString(env, formatted_url);
+  return j_formatted_url.Release();
+}
+
 }  // namespace android
 
 bool RegisterDomDistillerTabUtils(JNIEnv* env) {
   return android::RegisterNativesImpl(env);
 }