Reduce buggy usage of the registry controlled domain service.

content/renderer/webpublicsuffixlist_impl.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 "content/renderer/webpublicsuffixlist_impl.h"
 
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
 
 namespace content {
 
 WebPublicSuffixListImpl::~WebPublicSuffixListImpl() {
 }
 
 size_t WebPublicSuffixListImpl::getPublicSuffixLength(
     const blink::WebString& host) {
-  size_t result = net::registry_controlled_domains::GetRegistryLength(
-      host.utf8(),
-      net::registry_controlled_domains::INCLUDE_UNKNOWN_REGISTRIES,
-      net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES);
+  size_t result =
+      net::registry_controlled_domains::GetCanonicalHostRegistryLength(

[Peter Kasting, 2016/10/22 05:04:18]

Nit: Kinda makes me nervous that this assumes |host| is canonical.  At the least
we should probably rename |host| to |canonical_host| and write that it must be
canonical in the declaration comment?  I wish OriginAccessEntry had more
comments to indicate that it was only expected to be constructed with
canonicalized hosts or something?

[brettw, 2016/10/24 21:45:23]

I added the comment, there is no parameter name because it's WebKit :(

+          host.utf8(),
+          net::registry_controlled_domains::INCLUDE_UNKNOWN_REGISTRIES,
+          net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES);
   return result ? result : host.length();
 }
 
 } // namespace content