Re-reland r99689: Refactor the ContextInfo struct from bindings_utils into a real class.

Pull functionality from bindings_utils and event_bindings in as methods. Simplify lifetime management.

BUG=
TEST=

Review URL: http://codereview.chromium.org/8008027

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102554 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 97 deletions

diff --git a/chrome/renderer/extensions/bindings_utils.h b/chrome/renderer/extensions/bindings_utils.h
deleted file mode 100644
index 9f92184..0000000
--- a/chrome/renderer/extensions/bindings_utils.h
+++ /dev/null
@@ -1,97 +0,0 @@
-// Copyright (c) 2011 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_BINDINGS_UTILS_H_
-#define CHROME_RENDERER_EXTENSIONS_BINDINGS_UTILS_H_
-#pragma once
-
-#include "base/memory/linked_ptr.h"
-#include "base/memory/singleton.h"
-#include "v8/include/v8.h"
-
-#include <list>
-#include <map>
-#include <string>
-
-class RenderView;
-
-namespace WebKit {
-class WebFrame;
-}
-
-namespace bindings_utils {
-
-// Contains information about a JavaScript context that is hosting extension
-// bindings.
-struct ContextInfo {
-  ContextInfo(v8::Persistent<v8::Context> context,
-              const std::string& extension_id,
-              WebKit::WebFrame* frame);
-  ~ContextInfo();
-
-  // Returns the web frame associated with this context. Can also return NULL if
-  // the context has been disassociated with the frame, and not GC'd yet.
-  WebKit::WebFrame* GetWebFrame() const;
-
-  // Returns the RenderView associated wit hthis context. Can also return NULL.
-  RenderView* GetRenderView() const;
-
-  v8::Persistent<v8::Context> context;
-
-  // The extension ID this context is associated with.
-  std::string extension_id;
-
-  // The frame the context is associated with. ContextInfo can outlive its
-  // frame, so this should not be dereferenced. Use GetWebFrame() instead for
-  // most cases. This is used for comparisons during unload when GetWebFrame()
-  // doesn't work.
-  void* unsafe_frame;
-
-  // A count of the number of events that are listening in this context. When
-  // this is zero, |context| will be a weak handle.
-  int num_connected_events;
-};
-typedef std::list< linked_ptr<ContextInfo> > ContextList;
-
-// Returns a mutable reference to the ContextList. Note: be careful using this.
-// Calling into javascript may result in the list being modified, so don't rely
-// on iterators remaining valid between calls to javascript.
-ContextList& GetContexts();
-
-// Returns the ContextInfo item that has the given context.
-ContextList::iterator FindContext(v8::Handle<v8::Context> context);
-
-// Returns the ContextInfo for the current v8 context.
-ContextInfo* GetInfoForCurrentContext();
-
-// Returns the 'chromeHidden' object for the specified context.
-v8::Handle<v8::Object> GetChromeHiddenForContext(
-    v8::Handle<v8::Context> context);
-
-// Contains info relevant to a pending API request.
-struct PendingRequest {
- public :
-  PendingRequest(v8::Persistent<v8::Context> context, const std::string& name);
-  ~PendingRequest();
-
-  v8::Persistent<v8::Context> context;
-  std::string name;
-};
-typedef std::map<int, linked_ptr<PendingRequest> > PendingRequestMap;
-
-// Returns a mutable reference to the PendingRequestMap.
-PendingRequestMap& GetPendingRequestMap();
-
-// Call the named javascript function with the given arguments in a context.
-// The function name should be reachable from the chromeHidden object, and can
-// be a sub-property like "Port.dispatchOnMessage". Returns the result of
-// the function call. If an exception is thrown an empty Handle will be
-// returned.
-v8::Handle<v8::Value> CallFunctionInContext(v8::Handle<v8::Context> context,
-    const std::string& function_name, int argc,
-    v8::Handle<v8::Value>* argv);
-
-}  // namespace bindings_utils
-
-#endif  // CHROME_RENDERER_EXTENSIONS_BINDINGS_UTILS_H_