Drag and dropping text, parsable as url

components/bookmarks/browser/bookmark_node_data_views.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 "components/bookmarks/browser/bookmark_node_data.h"
 
 #include "base/logging.h"
 #include "base/pickle.h"
 #include "base/strings/utf_string_conversions.h"
+#include "ui/base/dragdrop/drag_and_drop_url_utils.h"
 #include "ui/base/dragdrop/os_exchange_data.h"
 #include "url/url_constants.h"
 
 namespace bookmarks {
 
 // static
 const ui::Clipboard::FormatType& BookmarkNodeData::GetBookmarkFormatType() {
   CR_DEFINE_STATIC_LOCAL(
       ui::Clipboard::FormatType,
       format,
@@ skipping 28 matching lines @@
   profile_path_.clear();
 
   if (data.HasCustomFormat(GetBookmarkFormatType())) {
     base::Pickle drag_data_pickle;
     if (data.GetPickledData(GetBookmarkFormatType(), &drag_data_pickle)) {
       if (!ReadFromPickle(&drag_data_pickle))
         return false;
     }
   } else {
     // See if there is a URL on the clipboard.
-    GURL url;
-    base::string16 title;
-    if (data.GetURLAndTitle(
-            ui::OSExchangeData::CONVERT_FILENAMES, &url, &title))
-      ReadFromTuple(url, title);
+    if (auto url_and_title = ui::TryToInterpretAsURL(

[dyaroshev, 2016/09/14 17:12:05]

No checks on the validity of url here, unlike other places. Do we want them
actually?

[Peter Kasting, 2016/09/15 21:26:53]

See other replies

+            data, ui::OSExchangeData::CONVERT_FILENAMES)) {
+      ReadFromTuple(url_and_title->first, url_and_title->second);
+    }
   }
 
   return is_valid();
 }
 
 }  // namespace bookmarks