Convert GURL::{host,path} to GURL::{host_piece,path_piece} for ==.

extensions/common/extension_urls.cc

Index: extensions/common/extension_urls.cc
diff --git a/extensions/common/extension_urls.cc b/extensions/common/extension_urls.cc
index 08fb32d62305b896ed32af8b4b16379fc7ea6d97..5f5d7935e7e89c3466d0eb57bdba4846c5ada264 100644
--- a/extensions/common/extension_urls.cc
+++ b/extensions/common/extension_urls.cc
@@ -84,8 +84,8 @@ bool IsWebstoreUpdateUrl(const GURL& update_url) {
   if (update_url == store_url) {
     return true;
   } else {

[Devlin, 2016/11/08 15:55:49]

Totally not your code, but since you're here... :)

Would you mind taking out this else (no need for else after a return) and
removing the parens around the conditions on the line below?

i.e.
if (update_url == store_url)
  return true;
return update_url.host_piece() == store_url.host_piece() &&
       update_url.path_piece() == store_url.path_piece();

[Charlie Harrison, 2016/11/08 16:00:55]

No problem. Done!

-    return (update_url.host() == store_url.host() &&
-            update_url.path() == store_url.path());
+    return (update_url.host_piece() == store_url.host_piece() &&
+            update_url.path_piece() == store_url.path_piece());
   }
 }