Use explicit WebString conversions in chrome/renderer/safe_browsing

chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc

Index: chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc
diff --git a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc
index a0ac57cd7c0efe7d51a2d90c6f0a7aebaa1559de..334cf5150024b47fefe82a7b4bfc1e9a2983a407 100644
--- a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc
+++ b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc
@@ -100,7 +100,7 @@ class TestPhishingDOMFeatureExtractor : public PhishingDOMFeatureExtractor {
       full_url = GURL(blink::WebStringToGURL(partial_url));
     } else if (!base_domain_.empty()) {
       // This is a partial URL and only one frame in testing html.
-      full_url = GURL("http://" + base_domain_).Resolve(partial_url);
+      full_url = GURL("http://" + base_domain_).Resolve(partial_url.utf16());

[Scott Hess - ex-Googler, 2017/01/24 17:08:46]

AFAICT, GURL::spec_ is UTF-8, though Resolve() can handle either UTF-8 or
UTF-16.  Since there is code above and below which calls partial_url.utf8(),
should this also be utf8()?

[kinuko, 2017/01/25 02:38:06]

Yeah makes sense, done.  I also changed GURL(partial_url.utf8()) on line 96 with
WebStringToGURL and changed the line 100 a simple assignment.

(Feel free to un-CQ if it doesn't look good or have newer owners make follow-up
changes!)

     } else {
       auto it = url_to_frame_domain_map_.find(partial_url.utf8());
       if (it != url_to_frame_domain_map_.end()) {