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

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

Issue 240603003: Remove ChromeV8Extension & most of ChromeV8Context (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cleanup Created 6 years, 8 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/renderer/extensions/chrome_v8_context_set.h
diff --git a/chrome/renderer/extensions/chrome_v8_context_set.h b/chrome/renderer/extensions/chrome_v8_context_set.h
deleted file mode 100644
index f47a61fe74a555d34ee8f0148eae68dc34f7404e..0000000000000000000000000000000000000000
--- a/chrome/renderer/extensions/chrome_v8_context_set.h
+++ /dev/null
@@ -1,90 +0,0 @@
-// Copyright (c) 2012 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.
-
-#ifndef CHROME_RENDERER_EXTENSIONS_CHROME_V8_CONTEXT_SET_H_
-#define CHROME_RENDERER_EXTENSIONS_CHROME_V8_CONTEXT_SET_H_
-
-#include <set>
-#include <string>
-
-#include "base/basictypes.h"
-#include "base/bind.h"
-#include "v8/include/v8.h"
-
-class GURL;
-
-namespace base {
-class ListValue;
-}
-
-namespace content {
-class RenderView;
-}
-
-namespace v8 {
-class Context;
-}
-
-namespace extensions {
-class ChromeV8Context;
-
-// A container of ExtensionBindingsContext. Since calling JavaScript within a
-// context can cause any number of contexts to be created or destroyed, this
-// has additional smarts to help with the set changing underneath callers.
-class ChromeV8ContextSet {
- public:
- ChromeV8ContextSet();
- ~ChromeV8ContextSet();
-
- int size() const;
-
- // Takes ownership of |context|.
- void Add(ChromeV8Context* context);
-
- // If the specified context is contained in this set, remove it, then delete
- // it asynchronously. After this call returns the context object will still
- // be valid, but its frame() pointer will be cleared.
- void Remove(ChromeV8Context* context);
-
- // Returns a copy to protect against changes.
- typedef std::set<ChromeV8Context*> ContextSet;
- ContextSet GetAll() const;
-
- // Gets the ChromeV8Context corresponding to v8::Context::GetCurrent(), or
- // NULL if no such context exists.
- ChromeV8Context* GetCurrent() const;
-
- // Gets the ChromeV8Context corresponding to v8::Context::GetCalling(), or
- // NULL if no such context exists.
- ChromeV8Context* GetCalling() const;
-
- // Gets the ChromeV8Context corresponding to the specified
- // v8::Context or NULL if no such context exists.
- ChromeV8Context* GetByV8Context(v8::Handle<v8::Context> context) const;
-
- // Synchronously runs |callback| with each ChromeV8Context that belongs to
- // |extension_id| in |render_view|.
- //
- // |extension_id| may be "" to match all extensions.
- // |render_view| may be NULL to match all render views.
- void ForEach(const std::string& extension_id,
- content::RenderView* render_view,
- const base::Callback<void(ChromeV8Context*)>& callback) const;
-
- // Cleans up contexts belonging to an unloaded extension.
- //
- // Returns the set of ChromeV8Contexts that were removed as a result. These
- // are safe to interact with until the end of the current event loop, since
- // they're deleted asynchronously.
- ContextSet OnExtensionUnloaded(const std::string& extension_id);
-
- private:
- ContextSet contexts_;
-
- DISALLOW_COPY_AND_ASSIGN(ChromeV8ContextSet);
-};
-
-} // namespace extensions
-
-#endif // CHROME_RENDERER_EXTENSIONS_CHROME_V8_CONTEXT_SET_H_

Powered by Google App Engine
This is Rietveld 408576698