Revert of Move the event attach/detach logic on unload from event.js to

Revert of Move the event attach/detach logic on unload from event.js to (patchset #7 id:130001 of https://codereview.chromium.org/1074273002/)

Reason for revert:
This is breaking ExtensionApiTest.EventsAreUnregistered on Win7 Dedubg bots.

See:
https://build.chromium.org/p/chromium.webkit/builders/Win7%20%28dbg%29/builds/16162

Original issue's description:
> Move the event attach/detach logic on unload from event.js to
> event_bindings.cc.
>
> This removes one of the reasons to call into JavaScript on context unload,
> which can crash. It's also more robust; it's confusing trying to maintain a
> data structure in JavaScript which reflects C++ state, which contributes to
> bugs like crbug.com/474718.
>
> Also clean up and formalise the script context invalidation system: CHECK
> rather than guarding against multiple invalidations, only invalidate script
> context related member variables, add an invalidation observer interface.
>
> BUG=474718, 475536
> R=rdevlin.cronin@chromium.org
>
> Committed: https://crrev.com/d99095034d2e88897ae82c8353f3327a3a1d03a5
> Cr-Commit-Position: refs/heads/master@{#324933}

TBR=rdevlin.cronin@chromium.org,kalman@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=474718, 475536
Committed: https://crrev.com/60a2857d182a7d980a7d43c9213cc1d96bf25f9e
Cr-Commit-Position: refs/heads/master@{#325050}

[mlamouri (slow - plz ping), 2015-04-14 10:21:30]

Created Revert of Move the event attach/detach logic on unload from event.js to

[commit-bot: I haz the power, 2015-04-14 10:22:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1083663004/1

[commit-bot: I haz the power, 2015-04-14 10:22:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-14 10:22:35]

Failed to apply patch for chrome/browser/extensions/events_apitest.cc:
While running git apply --index -3 -p1;
  error: patch failed: chrome/browser/extensions/events_apitest.cc:4
  Falling back to three-way merge...
  Applied patch to 'chrome/browser/extensions/events_apitest.cc' with conflicts.
  U chrome/browser/extensions/events_apitest.cc

Patch:       chrome/browser/extensions/events_apitest.cc
Index: chrome/browser/extensions/events_apitest.cc
diff --git a/chrome/browser/extensions/events_apitest.cc
b/chrome/browser/extensions/events_apitest.cc
index
d89971fa04904785088a9ab768a74188d6b71b20..3420022dc8b3ba664a153583b7ed8d65c89ae699
100644
--- a/chrome/browser/extensions/events_apitest.cc
+++ b/chrome/browser/extensions/events_apitest.cc
@@ -4,44 +4,6 @@
 
 #include "chrome/browser/extensions/extension_apitest.h"
 
-#include "extensions/browser/event_router.h"
-#include "extensions/browser/extension_registry.h"
-
-namespace extensions {
-
 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, Events) {
   ASSERT_TRUE(RunExtensionTest("events")) << message_;
 }
-
-// Tests that events are unregistered when an extension page shuts down.
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, EventsAreUnregistered) {
-  // In this test, page1.html registers for a number of events, then navigates
-  // to page2.html, which should unregister those events. page2.html notifies
-  // pass, by which point the event should have been unregistered.
-  EventRouter* event_router = EventRouter::Get(profile());
-  ExtensionRegistry* registry = ExtensionRegistry::Get(profile());
-
-  std::string test_extension_name = "events_are_unregistered";
-  ASSERT_TRUE(RunExtensionSubtest(test_extension_name, "page1.html"))
-      << message_;
-
-  // Find the extension we just installed by looking for the path.
-  const Extension* extension =
-      GetExtensionByPath(registry->enabled_extensions(),
-                         test_data_dir_.AppendASCII(test_extension_name));
-  ASSERT_TRUE(extension);
-  std::string id = extension->id();
-
-  // The page has closed, so no matter what all events are no longer listened
-  // to.
-  EXPECT_FALSE(
-      event_router->ExtensionHasEventListener(id, "browserAction.onClicked"));
-  EXPECT_FALSE(
-      event_router->ExtensionHasEventListener(id, "runtime.onStartup"));
-  EXPECT_FALSE(
-      event_router->ExtensionHasEventListener(id, "runtime.onSuspend"));
-  EXPECT_FALSE(
-      event_router->ExtensionHasEventListener(id, "runtime.onInstalled"));
-}
-
-}  // namespace extensions

[commit-bot: I haz the power, 2015-04-14 10:22:52]

The CQ bit was unchecked by commit-bot@chromium.org

[mlamouri (slow - plz ping), 2015-04-14 15:20:53]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2015-04-14 15:21:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1083663004/1

[commit-bot: I haz the power, 2015-04-14 15:22:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-04-14 15:23:47]

Patchset 1 (id:??) landed as
https://crrev.com/60a2857d182a7d980a7d43c9213cc1d96bf25f9e
Cr-Commit-Position: refs/heads/master@{#325050}