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

Unified Diff: include/v8.h

Issue 582953002: Revert "Require V8 to be explicitly initialized before an Isolate is created" (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 3 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 | « no previous file | samples/lineprocessor.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index 26d87102aacd58658c98982efb0b7b805209e603..e4c7087d304d8d88da2b7f12a7c903566021805f 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -4344,12 +4344,13 @@ typedef void (*JitCodeEventHandler)(const JitCodeEvent* event);
/**
- * Isolate represents an isolated instance of the V8 engine. V8 isolates have
- * completely separate states. Objects from one isolate must not be used in
- * other isolates. The embedder can create multiple isolates and use them in
- * parallel in multiple threads. An isolate can be entered by at most one
- * thread at any given time. The Locker/Unlocker API must be used to
- * synchronize.
+ * Isolate represents an isolated instance of the V8 engine. V8
+ * isolates have completely separate states. Objects from one isolate
+ * must not be used in other isolates. When V8 is initialized a
+ * default isolate is implicitly created and entered. The embedder
+ * can create additional isolates and use them in parallel in multiple
+ * threads. An isolate can be entered by at most one thread at any
+ * given time. The Locker/Unlocker API must be used to synchronize.
*/
class V8_EXPORT Isolate {
public:
@@ -4357,10 +4358,7 @@ class V8_EXPORT Isolate {
* Initial configuration parameters for a new Isolate.
*/
struct CreateParams {
- CreateParams()
- : entry_hook(NULL),
- code_event_handler(NULL),
- enable_serializer(false) {}
+ CreateParams() : entry_hook(NULL), code_event_handler(NULL) {}
/**
* The optional entry_hook allows the host application to provide the
@@ -4381,11 +4379,6 @@ class V8_EXPORT Isolate {
* ResourceConstraints to use for the new Isolate.
*/
ResourceConstraints constraints;
-
- /**
- * This flag currently renders the Isolate unusable.
- */
- bool enable_serializer;
};
@@ -4496,8 +4489,6 @@ class V8_EXPORT Isolate {
*
* When an isolate is no longer used its resources should be freed
* by calling Dispose(). Using the delete operator is not allowed.
- *
- * V8::Initialize() must have run prior to this.
*/
static Isolate* New(const CreateParams& params = CreateParams());
@@ -5111,8 +5102,9 @@ class V8_EXPORT V8 {
static void RemoveMemoryAllocationCallback(MemoryAllocationCallback callback);
/**
- * Initializes V8. This function needs to be called before the first Isolate
- * is created. It always returns true.
+ * Initializes from snapshot if possible. Otherwise, attempts to
+ * initialize from scratch. This function is called implicitly if
+ * you use the API without calling it first.
*/
static bool Initialize();
« no previous file with comments | « no previous file | samples/lineprocessor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698