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

Side by Side Diff: third_party/WebKit/Source/bindings/core/v8/V8EventListener.cpp

Issue 2386173002: reflow comments in Source/bindings/core/v8 (Closed)
Patch Set: Created 4 years, 2 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
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 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 if (handlerFunction.IsEmpty() || receiver.IsEmpty()) 87 if (handlerFunction.IsEmpty() || receiver.IsEmpty())
88 return v8::Local<v8::Value>(); 88 return v8::Local<v8::Value>();
89 89
90 if (!scriptState->getExecutionContext()->isDocument()) 90 if (!scriptState->getExecutionContext()->isDocument())
91 return v8::Local<v8::Value>(); 91 return v8::Local<v8::Value>();
92 92
93 LocalFrame* frame = toDocument(scriptState->getExecutionContext())->frame(); 93 LocalFrame* frame = toDocument(scriptState->getExecutionContext())->frame();
94 if (!frame) 94 if (!frame)
95 return v8::Local<v8::Value>(); 95 return v8::Local<v8::Value>();
96 96
97 // TODO(jochen): Consider moving this check into canExecuteScripts. http://crb ug.com/608641 97 // TODO(jochen): Consider moving this check into canExecuteScripts.
98 // http://crbug.com/608641
98 if (scriptState->world().isMainWorld() && 99 if (scriptState->world().isMainWorld() &&
99 !frame->script().canExecuteScripts(AboutToExecuteScript)) 100 !frame->script().canExecuteScripts(AboutToExecuteScript))
100 return v8::Local<v8::Value>(); 101 return v8::Local<v8::Value>();
101 102
102 v8::Local<v8::Value> parameters[1] = {jsEvent}; 103 v8::Local<v8::Value> parameters[1] = {jsEvent};
103 v8::Local<v8::Value> result; 104 v8::Local<v8::Value> result;
104 if (!V8ScriptRunner::callFunction(handlerFunction, frame->document(), 105 if (!V8ScriptRunner::callFunction(handlerFunction, frame->document(),
105 receiver, WTF_ARRAY_LENGTH(parameters), 106 receiver, WTF_ARRAY_LENGTH(parameters),
106 parameters, scriptState->isolate()) 107 parameters, scriptState->isolate())
107 .ToLocal(&result)) 108 .ToLocal(&result))
108 return v8::Local<v8::Value>(); 109 return v8::Local<v8::Value>();
109 return result; 110 return result;
110 } 111 }
111 112
112 } // namespace blink 113 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698