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

Unified Diff: chrome/renderer/extensions/chrome_v8_context.h

Issue 9653022: Revert 125811 - Convert app_bindings.js to the schema_generated_bindings.js infrastructure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 9 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
« no previous file with comments | « chrome/renderer/extensions/app_bindings.cc ('k') | chrome/renderer/extensions/chrome_v8_context.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/extensions/chrome_v8_context.h
===================================================================
--- chrome/renderer/extensions/chrome_v8_context.h (revision 125812)
+++ chrome/renderer/extensions/chrome_v8_context.h (working copy)
@@ -9,7 +9,6 @@
#include <string>
#include "base/basictypes.h"
-#include "chrome/common/extensions/feature.h"
#include "chrome/renderer/module_system.h"
#include "v8/include/v8.h"
@@ -28,10 +27,19 @@
// we won't need this object and it's a bit less state to keep track of.
class ChromeV8Context {
public:
+ enum ContextType {
+ CONTENT_SCRIPT,
+
+ // TODO(kalman): for now, have this as OTHER, since we only currently need
+ // know whether something is a content script or not. However, when
+ // necessary this should enumerate the other types, such as FRAME.
+ OTHER
+ };
+
ChromeV8Context(v8::Handle<v8::Context> context,
WebKit::WebFrame* frame,
const std::string& extension_id,
- extensions::Feature::Context context_type);
+ ContextType context_type);
~ChromeV8Context();
v8::Handle<v8::Context> v8_context() const {
@@ -49,7 +57,7 @@
web_frame_ = NULL;
}
- extensions::Feature::Context context_type() const {
+ ContextType context_type() const {
return context_type_;
}
@@ -108,7 +116,7 @@
std::string extension_id_;
// The type of context.
- extensions::Feature::Context context_type_;
+ ContextType context_type_;
// Owns and structures the JS that is injected to set up extension bindings.
scoped_ptr<ModuleSystem> module_system_;
« no previous file with comments | « chrome/renderer/extensions/app_bindings.cc ('k') | chrome/renderer/extensions/chrome_v8_context.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698