IDL compiler: move EventHandler null check out of generated bindings

Source/bindings/v8/V8EventListenerList.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 24 matching lines @@
 #include "bindings/v8/V8Binding.h"
 #include "bindings/v8/V8WorkerGlobalScopeEventListener.h"
 
 namespace WebCore {
 
 PassRefPtr<EventListener> V8EventListenerList::getEventListener(v8::Local<v8::Value> value, bool isAttribute, ListenerLookupType lookup)
 {
     v8::Handle<v8::Context> context = v8::Context::GetCurrent();
     if (context.IsEmpty())
         return 0;
+    v8::Isolate* isolate = context->GetIsolate();

[Nils Barth (inactive), 2013/10/22 01:46:38]

We should actually just pass an isolate in here, AFAICT;
we always have an isolate at the call site.
(For now making a local variable for clarity.)

     if (lookup == ListenerFindOnly)
-        return V8EventListenerList::findWrapper(value, isAttribute, context->GetIsolate());
+        return V8EventListenerList::findWrapper(value, isAttribute, isolate);
     if (V8DOMWrapper::isWrapperOfType(toInnerGlobalObject(context), &V8Window::info))
-        return V8EventListenerList::findOrCreateWrapper<V8EventListener>(value, isAttribute, context->GetIsolate());
-    return V8EventListenerList::findOrCreateWrapper<V8WorkerGlobalScopeEventListener>(value, isAttribute, context->GetIsolate());
+        return V8EventListenerList::findOrCreateWrapper<V8EventListener>(value, isAttribute, isolate);
+    return V8EventListenerList::findOrCreateWrapper<V8WorkerGlobalScopeEventListener>(value, isAttribute, isolate);
 }
 
 } // namespace WebCore