browser-plugin: Send a JSON string for the detail data with the events.

content/renderer/browser_plugin/browser_plugin_browsertest.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 "content/renderer/browser_plugin/browser_plugin_browsertest.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "content/common/browser_plugin_messages.h"
@@ skipping 256 matching lines @@
   // Send an event and verify that the event is deported.
   browser_plugin->handleInputEvent(WebKit::WebMouseEvent(),
                                    cursor_info);
   EXPECT_TRUE(browser_plugin_manager()->sink().GetUniqueMessageMatching(
       BrowserPluginHostMsg_HandleInputEvent::ID));
   browser_plugin_manager()->sink().ClearMessages();
 
   const char* kAddEventListener =
     "var msg;"
     "function exitListener(e) {"
-    "  msg = e.detail.reason;"
+    "  msg = JSON.parse(e.detail).reason;"
     "}"
     "document.getElementById('browserplugin')."
     "    addEventListener('-internal-exit', exitListener);";
 
   ExecuteJavaScript(kAddEventListener);
 
   // Pretend that the guest has terminated normally.
   browser_plugin->GuestGone(0, base::TERMINATION_STATUS_NORMAL_TERMINATION);
 
   // Verify that our event listener has fired.
@@ skipping 19 matching lines @@
   ExecuteJavaScript("x = document.getElementById('browserplugin'); "
                     "x.parentNode.removeChild(x);");
   ProcessPendingMessages();
   EXPECT_TRUE(browser_plugin_manager()->sink().GetUniqueMessageMatching(
       BrowserPluginHostMsg_PluginDestroyed::ID));
 }
 
 TEST_F(BrowserPluginTest, CustomEvents) {
   const char* kAddEventListener =
     "var url;"
-    "function nav(u) {"
-    "  url = u.detail.url;"
+    "function nav(e) {"
+    "  url = JSON.parse(e.detail).url;"
     "}"
     "document.getElementById('browserplugin')."
     "    addEventListener('-internal-loadcommit', nav);";
   const char* kRemoveEventListener =
     "document.getElementById('browserplugin')."
     "    removeEventListener('-internal-loadcommit', nav);";
   const char* kGetProcessID =
       "document.getElementById('browserplugin').getProcessId()";
   const char* kGoogleURL = "http://www.google.com/";
   const char* kGoogleNewsURL = "http://news.google.com/";
@@ skipping 165 matching lines @@
   partition_value = ExecuteScriptAndReturnString(
       "document.getElementById('browserplugin').partition");
   EXPECT_STREQ("storage", partition_value.c_str());
 }
 
 // This test verifies that we can mutate the event listener vector
 // within an event listener.
 TEST_F(BrowserPluginTest, RemoveEventListenerInEventListener) {
   const char* kAddEventListener =
     "var url;"
-    "function nav(u) {"
-    "  url = u.detail.url;"
+    "function nav(e) {"
+    "  url = JSON.parse(e.detail).url;"
     "  document.getElementById('browserplugin')."
     "      removeEventListener('-internal-loadcommit', nav);"
     "}"
     "document.getElementById('browserplugin')."
     "    addEventListener('-internal-loadcommit', nav);";
   const char* kGoogleURL = "http://www.google.com/";
   const char* kGoogleNewsURL = "http://news.google.com/";
   const char* kGetProcessID =
       "document.getElementById('browserplugin').getProcessId()";
 
@@ skipping 93 matching lines @@
   ASSERT_TRUE(IPC::ReadParam(msg, &iter, &resize_params));
   int instance_id = resize_params.a;
 
   MockBrowserPlugin* browser_plugin =
       static_cast<MockBrowserPlugin*>(
           browser_plugin_manager()->GetBrowserPlugin(instance_id));
   ASSERT_TRUE(browser_plugin);
 
   const char* kAddEventListener =
     "function exitListener(e) {"
-    "  if (e.detail.reason == 'killed') {"
+    "  if (JSON.parse(e.detail).reason == 'killed') {"
     "    var bp = document.getElementById('browserplugin');"
     "    bp.parentNode.removeChild(bp);"
     "  }"
     "}"
     "document.getElementById('browserplugin')."
     "    addEventListener('-internal-exit', exitListener);";
 
   ExecuteJavaScript(kAddEventListener);
 
   // Pretend that the guest has crashed.
@@ skipping 60 matching lines @@
         &params);
      EXPECT_FALSE(params.enable);
      EXPECT_EQ(42, params.min_width);
      EXPECT_EQ(43, params.min_height);
      EXPECT_EQ(1337, params.max_width);
      EXPECT_EQ(1338, params.max_height);
   }
 }
 
 }  // namespace content