Remove unnecessary calls to GURL()

chrome/browser/bookmarks/managed_bookmark_service_unittest.cc

Index: chrome/browser/bookmarks/managed_bookmark_service_unittest.cc
diff --git a/chrome/browser/bookmarks/managed_bookmark_service_unittest.cc b/chrome/browser/bookmarks/managed_bookmark_service_unittest.cc
index 7d95cc37484213a63892e0e310a7df3b57f89704..1dec842054cb19538cbf63257a56ec46a24c4104 100644
--- a/chrome/browser/bookmarks/managed_bookmark_service_unittest.cc
+++ b/chrome/browser/bookmarks/managed_bookmark_service_unittest.cc
@@ -126,7 +126,7 @@ class ManagedBookmarkServiceTest : public testing::Test {
       }
     } else if (node->is_url()) {
       std::string url;
-      if (!dict->GetString("url", &url) || node->url() != GURL(url))
+      if (!dict->GetString("url", &url) || node->url() != url)

[Charlie Harrison, 2016/11/09 21:12:58]

Will |url| always be canonicalized here?

[cfredric, 2016/11/09 22:12:46]

Currently, yes. The only usage of DictionaryValue::SetString with a key of "url"
uses GURL(url).spec() as the value.

         return false;
     } else {
       return false;