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

Side by Side Diff: src/messages.cc

Issue 8391045: Handlify the remaining CallStubCompiler functions. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 9 years, 1 month 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 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 if (isolate->has_scheduled_exception()) { 119 if (isolate->has_scheduled_exception()) {
120 isolate->clear_scheduled_exception(); 120 isolate->clear_scheduled_exception();
121 } 121 }
122 } else { 122 } else {
123 for (int i = 0; i < global_length; i++) { 123 for (int i = 0; i < global_length; i++) {
124 HandleScope scope; 124 HandleScope scope;
125 if (global_listeners.get(i)->IsUndefined()) continue; 125 if (global_listeners.get(i)->IsUndefined()) continue;
126 v8::NeanderObject listener(JSObject::cast(global_listeners.get(i))); 126 v8::NeanderObject listener(JSObject::cast(global_listeners.get(i)));
127 Handle<Foreign> callback_obj(Foreign::cast(listener.get(0))); 127 Handle<Foreign> callback_obj(Foreign::cast(listener.get(0)));
128 v8::MessageCallback callback = 128 v8::MessageCallback callback =
129 FUNCTION_CAST<v8::MessageCallback>(callback_obj->address()); 129 FUNCTION_CAST<v8::MessageCallback>(callback_obj->foreign_address());
130 Handle<Object> callback_data(listener.get(1)); 130 Handle<Object> callback_data(listener.get(1));
131 { 131 {
132 // Do not allow exceptions to propagate. 132 // Do not allow exceptions to propagate.
133 v8::TryCatch try_catch; 133 v8::TryCatch try_catch;
134 callback(api_message_obj, v8::Utils::ToLocal(callback_data)); 134 callback(api_message_obj, v8::Utils::ToLocal(callback_data));
135 } 135 }
136 if (isolate->has_scheduled_exception()) { 136 if (isolate->has_scheduled_exception()) {
137 isolate->clear_scheduled_exception(); 137 isolate->clear_scheduled_exception();
138 } 138 }
139 } 139 }
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 173
174 174
175 SmartArrayPointer<char> MessageHandler::GetLocalizedMessage( 175 SmartArrayPointer<char> MessageHandler::GetLocalizedMessage(
176 Handle<Object> data) { 176 Handle<Object> data) {
177 HandleScope scope; 177 HandleScope scope;
178 return GetMessage(data)->ToCString(DISALLOW_NULLS); 178 return GetMessage(data)->ToCString(DISALLOW_NULLS);
179 } 179 }
180 180
181 181
182 } } // namespace v8::internal 182 } } // namespace v8::internal
OLDNEW
« src/arm/stub-cache-arm.cc ('K') | « src/ia32/stub-cache-ia32.cc ('k') | src/objects.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698