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

Side by Side Diff: extensions/renderer/dom_activity_logger.cc

Issue 672403002: Cleanup: Remove most enable_extensions logic in extensions code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix android Created 6 years, 1 month 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/renderer/dispatcher.cc ('k') | extensions/renderer/messaging_bindings.cc » ('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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/renderer/dom_activity_logger.h" 5 #include "extensions/renderer/dom_activity_logger.h"
6 6
7 #include "content/public/renderer/render_thread.h" 7 #include "content/public/renderer/render_thread.h"
8 #include "content/public/renderer/v8_value_converter.h" 8 #include "content/public/renderer/v8_value_converter.h"
9 #include "extensions/common/dom_action_types.h" 9 #include "extensions/common/dom_action_types.h"
10 #include "extensions/common/extension_messages.h" 10 #include "extensions/common/extension_messages.h"
(...skipping 29 matching lines...) Expand all
40 } // namespace 40 } // namespace
41 41
42 DOMActivityLogger::DOMActivityLogger(const std::string& extension_id) 42 DOMActivityLogger::DOMActivityLogger(const std::string& extension_id)
43 : extension_id_(extension_id) { 43 : extension_id_(extension_id) {
44 } 44 }
45 45
46 DOMActivityLogger::~DOMActivityLogger() {} 46 DOMActivityLogger::~DOMActivityLogger() {}
47 47
48 void DOMActivityLogger::AttachToWorld(int world_id, 48 void DOMActivityLogger::AttachToWorld(int world_id,
49 const std::string& extension_id) { 49 const std::string& extension_id) {
50 #if defined(ENABLE_EXTENSIONS)
51 // If there is no logger registered for world_id, construct a new logger 50 // If there is no logger registered for world_id, construct a new logger
52 // and register it with world_id. 51 // and register it with world_id.
53 if (!blink::hasDOMActivityLogger(world_id, 52 if (!blink::hasDOMActivityLogger(world_id,
54 WebString::fromUTF8(extension_id))) { 53 WebString::fromUTF8(extension_id))) {
55 DOMActivityLogger* logger = new DOMActivityLogger(extension_id); 54 DOMActivityLogger* logger = new DOMActivityLogger(extension_id);
56 blink::setDOMActivityLogger(world_id, 55 blink::setDOMActivityLogger(world_id,
57 WebString::fromUTF8(extension_id), 56 WebString::fromUTF8(extension_id),
58 logger); 57 logger);
59 } 58 }
60 #endif
61 } 59 }
62 60
63 void DOMActivityLogger::logGetter(const WebString& api_name, 61 void DOMActivityLogger::logGetter(const WebString& api_name,
64 const WebURL& url, 62 const WebURL& url,
65 const WebString& title) { 63 const WebString& title) {
66 SendDomActionMessage(api_name.utf8(), 64 SendDomActionMessage(api_name.utf8(),
67 url, 65 url,
68 title, 66 title,
69 DomActionType::GETTER, 67 DomActionType::GETTER,
70 scoped_ptr<base::ListValue>(new base::ListValue())); 68 scoped_ptr<base::ListValue>(new base::ListValue()));
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
126 params.api_call = api_call; 124 params.api_call = api_call;
127 params.url = url; 125 params.url = url;
128 params.url_title = url_title; 126 params.url_title = url_title;
129 params.call_type = call_type; 127 params.call_type = call_type;
130 params.arguments.Swap(args.get()); 128 params.arguments.Swap(args.get());
131 content::RenderThread::Get()->Send( 129 content::RenderThread::Get()->Send(
132 new ExtensionHostMsg_AddDOMActionToActivityLog(extension_id_, params)); 130 new ExtensionHostMsg_AddDOMActionToActivityLog(extension_id_, params));
133 } 131 }
134 132
135 } // namespace extensions 133 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/renderer/dispatcher.cc ('k') | extensions/renderer/messaging_bindings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698