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

Unified Diff: chrome/browser/extensions/extension_activity_log.cc

Issue 10824198: Move small c/b/extensions classes into extensions namespace no.1 (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Latest-er master for cq Created 8 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_activity_log.cc
diff --git a/chrome/browser/extensions/extension_activity_log.cc b/chrome/browser/extensions/extension_activity_log.cc
deleted file mode 100644
index ea9a0486b5d55868c98b0c9f20cad21e745adaaa..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/extension_activity_log.cc
+++ /dev/null
@@ -1,83 +0,0 @@
-// Copyright (c) 2011 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 "chrome/browser/extensions/extension_activity_log.h"
-
-#include "base/command_line.h"
-#include "base/logging.h"
-#include "chrome/common/chrome_switches.h"
-#include "chrome/common/extensions/extension.h"
-
-ExtensionActivityLog::ExtensionActivityLog() {
- log_activity_to_stdout_ = CommandLine::ForCurrentProcess()->
- HasSwitch(switches::kEnableExtensionActivityLogging);
-}
-
-ExtensionActivityLog::~ExtensionActivityLog() {
-}
-
-// static
-ExtensionActivityLog* ExtensionActivityLog::GetInstance() {
- return Singleton<ExtensionActivityLog>::get();
-}
-
-void ExtensionActivityLog::AddObserver(
- const extensions::Extension* extension,
- ExtensionActivityLog::Observer* observer) {
- base::AutoLock scoped_lock(lock_);
-
- if (observers_.count(extension) == 0) {
- observers_[extension] = new ObserverListThreadSafe<Observer>;
- }
-
- observers_[extension]->AddObserver(observer);
-}
-
-void ExtensionActivityLog::RemoveObserver(
- const extensions::Extension* extension,
- ExtensionActivityLog::Observer* observer) {
- base::AutoLock scoped_lock(lock_);
-
- if (observers_.count(extension) == 1) {
- observers_[extension]->RemoveObserver(observer);
- }
-}
-// Extension*
-bool ExtensionActivityLog::HasObservers(
- const extensions::Extension* extension) const {
- base::AutoLock scoped_lock(lock_);
-
- // We also return true if extension activity logging is enabled since in that
- // case this class is observing all extensions.
- return observers_.count(extension) > 0 || log_activity_to_stdout_;
-}
-
-void ExtensionActivityLog::Log(const extensions::Extension* extension,
- Activity activity,
- const std::string& msg) const {
- base::AutoLock scoped_lock(lock_);
-
- ObserverMap::const_iterator iter = observers_.find(extension);
- if (iter != observers_.end()) {
- iter->second->Notify(&Observer::OnExtensionActivity, extension, activity,
- msg);
- }
-
- if (log_activity_to_stdout_) {
- LOG(INFO) << extension->id() + ":" + ActivityToString(activity) + ":" + msg;
- }
-}
-
-// static
-const char* ExtensionActivityLog::ActivityToString(Activity activity) {
- switch (activity) {
- case ExtensionActivityLog::ACTIVITY_EXTENSION_API_CALL:
- return "api_call";
- case ExtensionActivityLog::ACTIVITY_EXTENSION_API_BLOCK:
- return "api_block";
- default:
- NOTREACHED();
- return "";
- }
-}
« no previous file with comments | « chrome/browser/extensions/extension_activity_log.h ('k') | chrome/browser/extensions/extension_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698