Revert 146038 as it might have broken chromeos browser_tests - Make eventArgumentMassagers asynchro…

chrome/renderer/extensions/event_unittest.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/test/base/module_system_test.h"
 
 #include "grit/renderer_resources.h"
 
 namespace {
 
@@ skipping 47 matching lines @@
         "exports.HasFilteredListener = function(name) {"
         "  return attachedFilteredListeners[name].length;"
         "};");
     OverrideNativeHandler("chrome_hidden",
         "var chromeHidden = {};"
         "exports.GetChromeHidden = function() { return chromeHidden; };");
     OverrideNativeHandler("sendRequest",
         "exports.sendRequest = function() {};");
     OverrideNativeHandler("apiDefinitions",
         "exports.GetExtensionAPIDefinition = function() {};");
-    OverrideNativeHandler("logging",
-        "exports.DCHECK = function() {};");
   }
 };
 
 TEST_F(EventUnittest, TestNothing) {
   ExpectNoAssertionsMade();
 }
 
 TEST_F(EventUnittest, AddRemoveTwoListeners) {
   ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
   RegisterModule("test",
@@ skipping 63 matching lines @@
 
 TEST_F(EventUnittest, NamedEventDispatch) {
   ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
   RegisterModule("test",
       "var event = require('event');"
       "var chromeHidden = requireNative('chrome_hidden').GetChromeHidden();"
       "var assert = requireNative('assert');"
       "var e = new event.Event('myevent');"
       "var called = false;"
       "e.addListener(function() { called = true; });"
-      "chromeHidden.Event.dispatchJSON('myevent', []);"
+      "chromeHidden.Event.dispatch('myevent', []);"
       "assert.AssertTrue(called);");
   module_system_->Require("test");
 }
 
 TEST_F(EventUnittest, AddListenerWithFiltersThrowsErrorByDefault) {
   ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
   RegisterModule("test",
       "var event = require('event');"
       "var assert = requireNative('assert');"
       "var e = new event.Event('myevent');"
@@ skipping 77 matching lines @@
       "var caught = false;"
       "try {"
       "  e.addListener(cb, filters);"
       "} catch (e) {"
       "  caught = true;"
       "}"
       "assert.AssertTrue(caught);");
   module_system_->Require("test");
 }
 
-TEST_F(EventUnittest, MaxListeners) {
-  ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
-  RegisterModule("test",
-      "var event = require('event');"
-      "var assert = requireNative('assert');"
-      "var eventOpts = {supportsListeners: true, maxListeners: 1};"
-      "var e = new event.Event('myevent', undefined, eventOpts);"
-      "var cb = function() {};"
-      "var caught = false;"
-      "try {"
-      "  e.addListener(cb);"
-      "} catch (e) {"
-      "  caught = true;"
-      "}"
-      "assert.AssertTrue(!caught);"
-      "try {"
-      "  e.addListener(cb);"
-      "} catch (e) {"
-      "  caught = true;"
-      "}"
-      "assert.AssertTrue(caught);");
-  module_system_->Require("test");
-}
-
 }  // namespace