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

Side by Side Diff: Source/bindings/v8/V8AbstractEventListener.h

Issue 209713003: Make DOMWrapperWorld::current() return a reference instead of a pointer (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008, 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008, 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 { 102 {
103 return !m_listener.isEmpty(); 103 return !m_listener.isEmpty();
104 } 104 }
105 105
106 void clearListenerObject() 106 void clearListenerObject()
107 { 107 {
108 m_listener.clear(); 108 m_listener.clear();
109 } 109 }
110 110
111 virtual bool belongsToTheCurrentWorld() const OVERRIDE FINAL; 111 virtual bool belongsToTheCurrentWorld() const OVERRIDE FINAL;
112 virtual DOMWrapperWorld* world() const OVERRIDE FINAL { return m_world.g et(); } 112 DOMWrapperWorld& world() const { return *m_world; }
113 v8::Isolate* isolate() const { return m_isolate; } 113 v8::Isolate* isolate() const { return m_isolate; }
114 114
115 protected: 115 protected:
116 V8AbstractEventListener(bool isAttribute, DOMWrapperWorld*, v8::Isolate* ); 116 V8AbstractEventListener(bool isAttribute, DOMWrapperWorld&, v8::Isolate* );
117 117
118 virtual void prepareListenerObject(ExecutionContext*) { } 118 virtual void prepareListenerObject(ExecutionContext*) { }
119 119
120 void setListenerObject(v8::Handle<v8::Object> listener); 120 void setListenerObject(v8::Handle<v8::Object> listener);
121 121
122 void invokeEventHandler(ExecutionContext*, Event*, v8::Local<v8::Value> jsEvent); 122 void invokeEventHandler(ExecutionContext*, Event*, v8::Local<v8::Value> jsEvent);
123 123
124 // Get the receiver object to use for event listener call. 124 // Get the receiver object to use for event listener call.
125 v8::Local<v8::Object> getReceiverObject(ExecutionContext*, Event*); 125 v8::Local<v8::Object> getReceiverObject(ExecutionContext*, Event*);
126 126
(...skipping 12 matching lines...) Expand all
139 // Indicates if this is an HTML type listener. 139 // Indicates if this is an HTML type listener.
140 bool m_isAttribute; 140 bool m_isAttribute;
141 141
142 RefPtr<DOMWrapperWorld> m_world; 142 RefPtr<DOMWrapperWorld> m_world;
143 v8::Isolate* m_isolate; 143 v8::Isolate* m_isolate;
144 }; 144 };
145 145
146 } // namespace WebCore 146 } // namespace WebCore
147 147
148 #endif // V8AbstractEventListener_h 148 #endif // V8AbstractEventListener_h
OLDNEW
« no previous file with comments | « Source/bindings/v8/SerializedScriptValue.cpp ('k') | Source/bindings/v8/V8AbstractEventListener.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698