Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(357)

Side by Side Diff: extensions/browser/event_router.cc

Issue 1201063002: Set up the infrastructure for Extension event metrics. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebaaaaase Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « extensions/browser/event_router.h ('k') | extensions/browser/extension_event_histogram_value.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/browser/event_router.h" 5 #include "extensions/browser/event_router.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/atomic_sequence_num.h" 9 #include "base/atomic_sequence_num.h"
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 776 matching lines...) Expand 10 before | Expand all | Expand 10 after
787 listeners_.LoadFilteredLazyListeners(extension->id(), *filtered_events); 787 listeners_.LoadFilteredLazyListeners(extension->id(), *filtered_events);
788 } 788 }
789 789
790 void EventRouter::OnExtensionUnloaded(content::BrowserContext* browser_context, 790 void EventRouter::OnExtensionUnloaded(content::BrowserContext* browser_context,
791 const Extension* extension, 791 const Extension* extension,
792 UnloadedExtensionInfo::Reason reason) { 792 UnloadedExtensionInfo::Reason reason) {
793 // Remove all registered listeners from our cache. 793 // Remove all registered listeners from our cache.
794 listeners_.RemoveListenersForExtension(extension->id()); 794 listeners_.RemoveListenersForExtension(extension->id());
795 } 795 }
796 796
797 Event::Event(const std::string& event_name, 797 Event::Event(events::HistogramValue histogram_value,
798 const std::string& event_name,
798 scoped_ptr<base::ListValue> event_args) 799 scoped_ptr<base::ListValue> event_args)
799 : event_name(event_name), 800 : histogram_value(histogram_value),
801 event_name(event_name),
800 event_args(event_args.Pass()), 802 event_args(event_args.Pass()),
801 restrict_to_browser_context(NULL), 803 restrict_to_browser_context(NULL),
802 user_gesture(EventRouter::USER_GESTURE_UNKNOWN) { 804 user_gesture(EventRouter::USER_GESTURE_UNKNOWN) {
803 DCHECK(this->event_args.get()); 805 DCHECK(this->event_args.get());
804 } 806 }
805 807
806 Event::Event(const std::string& event_name, 808 Event::Event(events::HistogramValue histogram_value,
809 const std::string& event_name,
807 scoped_ptr<base::ListValue> event_args, 810 scoped_ptr<base::ListValue> event_args,
808 BrowserContext* restrict_to_browser_context) 811 BrowserContext* restrict_to_browser_context)
809 : event_name(event_name), 812 : histogram_value(histogram_value),
813 event_name(event_name),
810 event_args(event_args.Pass()), 814 event_args(event_args.Pass()),
811 restrict_to_browser_context(restrict_to_browser_context), 815 restrict_to_browser_context(restrict_to_browser_context),
812 user_gesture(EventRouter::USER_GESTURE_UNKNOWN) { 816 user_gesture(EventRouter::USER_GESTURE_UNKNOWN) {
813 DCHECK(this->event_args.get()); 817 DCHECK(this->event_args.get());
814 } 818 }
815 819
816 Event::Event(const std::string& event_name, 820 Event::Event(events::HistogramValue histogram_value,
821 const std::string& event_name,
817 scoped_ptr<ListValue> event_args, 822 scoped_ptr<ListValue> event_args,
818 BrowserContext* restrict_to_browser_context, 823 BrowserContext* restrict_to_browser_context,
819 const GURL& event_url, 824 const GURL& event_url,
820 EventRouter::UserGestureState user_gesture, 825 EventRouter::UserGestureState user_gesture,
821 const EventFilteringInfo& filter_info) 826 const EventFilteringInfo& filter_info)
822 : event_name(event_name), 827 : histogram_value(histogram_value),
828 event_name(event_name),
823 event_args(event_args.Pass()), 829 event_args(event_args.Pass()),
824 restrict_to_browser_context(restrict_to_browser_context), 830 restrict_to_browser_context(restrict_to_browser_context),
825 event_url(event_url), 831 event_url(event_url),
826 user_gesture(user_gesture), 832 user_gesture(user_gesture),
827 filter_info(filter_info) { 833 filter_info(filter_info) {
828 DCHECK(this->event_args.get()); 834 DCHECK(this->event_args.get());
829 } 835 }
830 836
831 Event::~Event() {} 837 Event::~Event() {}
832 838
833 Event* Event::DeepCopy() { 839 Event* Event::DeepCopy() {
834 Event* copy = new Event(event_name, 840 Event* copy = new Event(histogram_value, event_name,
835 scoped_ptr<base::ListValue>(event_args->DeepCopy()), 841 scoped_ptr<base::ListValue>(event_args->DeepCopy()),
836 restrict_to_browser_context, 842 restrict_to_browser_context, event_url, user_gesture,
837 event_url,
838 user_gesture,
839 filter_info); 843 filter_info);
840 copy->will_dispatch_callback = will_dispatch_callback; 844 copy->will_dispatch_callback = will_dispatch_callback;
841 return copy; 845 return copy;
842 } 846 }
843 847
844 EventListenerInfo::EventListenerInfo(const std::string& event_name, 848 EventListenerInfo::EventListenerInfo(const std::string& event_name,
845 const std::string& extension_id, 849 const std::string& extension_id,
846 const GURL& listener_url, 850 const GURL& listener_url,
847 content::BrowserContext* browser_context) 851 content::BrowserContext* browser_context)
848 : event_name(event_name), 852 : event_name(event_name),
849 extension_id(extension_id), 853 extension_id(extension_id),
850 listener_url(listener_url), 854 listener_url(listener_url),
851 browser_context(browser_context) { 855 browser_context(browser_context) {
852 } 856 }
853 857
854 } // namespace extensions 858 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/browser/event_router.h ('k') | extensions/browser/extension_event_histogram_value.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698