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

Side by Side Diff: third_party/WebKit/Source/platform/v8_inspector/V8InjectedScriptHost.h

Issue 2246233002: [DevTools] Move platform/v8_inspector classes under v8_inspector namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef V8InjectedScriptHost_h 5 #ifndef V8InjectedScriptHost_h
6 #define V8InjectedScriptHost_h 6 #define V8InjectedScriptHost_h
7 7
8 #include "platform/inspector_protocol/InspectorProtocol.h" 8 #include "platform/inspector_protocol/InspectorProtocol.h"
9 #include <v8.h> 9 #include <v8.h>
10 10
11 namespace blink { 11 namespace v8_inspector {
12 12
13 class V8InspectorImpl; 13 class V8InspectorImpl;
14 14
15 namespace protocol = blink::protocol;
16
15 // SECURITY NOTE: Although the InjectedScriptHost is intended for use solely by the inspector, 17 // SECURITY NOTE: Although the InjectedScriptHost is intended for use solely by the inspector,
16 // a reference to the InjectedScriptHost may be leaked to the page being inspect ed. Thus, the 18 // a reference to the InjectedScriptHost may be leaked to the page being inspect ed. Thus, the
17 // InjectedScriptHost must never implemment methods that have more power over th e page than the 19 // InjectedScriptHost must never implemment methods that have more power over th e page than the
18 // page already has itself (e.g. origin restriction bypasses). 20 // page already has itself (e.g. origin restriction bypasses).
19 21
20 class V8InjectedScriptHost { 22 class V8InjectedScriptHost {
21 public: 23 public:
22 // We expect that debugger outlives any JS context and thus V8InjectedScript Host (owned by JS) 24 // We expect that debugger outlives any JS context and thus V8InjectedScript Host (owned by JS)
23 // is destroyed before inspector. 25 // is destroyed before inspector.
24 static v8::Local<v8::Object> create(v8::Local<v8::Context>, V8InspectorImpl* ); 26 static v8::Local<v8::Object> create(v8::Local<v8::Context>, V8InspectorImpl* );
25 private: 27 private:
26 static void internalConstructorNameCallback(const v8::FunctionCallbackInfo<v 8::Value>&); 28 static void internalConstructorNameCallback(const v8::FunctionCallbackInfo<v 8::Value>&);
27 static void formatAccessorsAsProperties(const v8::FunctionCallbackInfo<v8::V alue>&); 29 static void formatAccessorsAsProperties(const v8::FunctionCallbackInfo<v8::V alue>&);
28 static void subtypeCallback(const v8::FunctionCallbackInfo<v8::Value>&); 30 static void subtypeCallback(const v8::FunctionCallbackInfo<v8::Value>&);
29 static void getInternalPropertiesCallback(const v8::FunctionCallbackInfo<v8: :Value>&); 31 static void getInternalPropertiesCallback(const v8::FunctionCallbackInfo<v8: :Value>&);
30 static void objectHasOwnPropertyCallback(const v8::FunctionCallbackInfo<v8:: Value>&); 32 static void objectHasOwnPropertyCallback(const v8::FunctionCallbackInfo<v8:: Value>&);
31 static void bindCallback(const v8::FunctionCallbackInfo<v8::Value>&); 33 static void bindCallback(const v8::FunctionCallbackInfo<v8::Value>&);
32 static void proxyTargetValueCallback(const v8::FunctionCallbackInfo<v8::Valu e>&); 34 static void proxyTargetValueCallback(const v8::FunctionCallbackInfo<v8::Valu e>&);
33 }; 35 };
34 36
35 } // namespace blink 37 } // namespace v8_inspector
36 38
37 #endif // V8InjectedScriptHost_h 39 #endif // V8InjectedScriptHost_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698