Only use host() for chrome:// urls in EventListener

extensions/browser/event_listener_map.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 "extensions/browser/event_listener_map.h"
 
 #include <stddef.h>
 
 #include <utility>
 
@@ skipping 19 matching lines @@
   return base::WrapUnique(new EventListener(event_name, extension_id, GURL(),
                                             process, std::move(filter)));
 }
 
 // static
 std::unique_ptr<EventListener> EventListener::ForURL(
     const std::string& event_name,
     const GURL& listener_url,
     content::RenderProcessHost* process,
     std::unique_ptr<base::DictionaryValue> filter) {
-  return base::WrapUnique(new EventListener(event_name, "", listener_url,
-                                            process, std::move(filter)));
+  // Use only the origin to identify the event listener, e.g. chrome://settings
+  // for chrome://settings/accounts, to avoid multiple events being triggered
+  // for the same process. See crbug.com/536858 for details.
+  return base::WrapUnique(new EventListener(
+      event_name, "", listener_url.GetOrigin(), process, std::move(filter)));

[Devlin, 2016/12/27 17:12:09]

for annoying reasons, let's actually make this
url::Origin(listener_url).GetURL().  Also, please add a
// TODO(devlin): If we dispatched events to processes more intelligently, this
could be avoided.

[stevenjb, 2016/12/27 17:57:37]

Done.

 }
 
 EventListener::~EventListener() {}
 
 bool EventListener::Equals(const EventListener* other) const {
   // We don't check matcher_id equality because we want a listener with a
   // filter that hasn't been added to EventFilter to match one that is
   // equivalent but has.
   return event_name_ == other->event_name_ &&
          extension_id_ == other->extension_id_ &&
@@ skipping 226 matching lines @@
     return;
   event_filter_.RemoveEventMatcher(listener->matcher_id());
   CHECK_EQ(1u, listeners_by_matcher_id_.erase(listener->matcher_id()));
 }
 
 bool EventListenerMap::IsFilteredEvent(const Event& event) const {
   return filtered_events_.count(event.event_name) > 0u;
 }
 
 }  // namespace extensions