Remove browser patching mechanism for handling javascript dialogs in chromium port
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Feb 2013 00:50:41 +0000 (00:50 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Feb 2013 00:50:41 +0000 (00:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110531

Patch by Ken Kania <kkania@chromium.org> on 2013-02-25
Reviewed by Pavel Feldman.

* public/WebDevToolsAgent.h:
* src/WebDevToolsAgentImpl.cpp:
(WebKit::WebDevToolsAgentImpl::handleJavaScriptDialog):
(WebKit::browserHintToString):
(WebKit::browserHintFromString):
(WebKit::WebDevToolsAgent::patchWithBrowserData):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@143987 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebDevToolsAgent.h
Source/WebKit/chromium/src/WebDevToolsAgentImpl.cpp

index 301ae63..04fb6ac 100644 (file)
@@ -1,3 +1,17 @@
+2013-02-25  Ken Kania  <kkania@chromium.org>
+
+        Remove browser patching mechanism for handling javascript dialogs in chromium port
+        https://bugs.webkit.org/show_bug.cgi?id=110531
+
+        Reviewed by Pavel Feldman.
+
+        * public/WebDevToolsAgent.h:
+        * src/WebDevToolsAgentImpl.cpp:
+        (WebKit::WebDevToolsAgentImpl::handleJavaScriptDialog):
+        (WebKit::browserHintToString):
+        (WebKit::browserHintFromString):
+        (WebKit::WebDevToolsAgent::patchWithBrowserData):
+
 2013-02-25  James Robinson  <jamesr@chromium.org>
 
         [chromium] Plumb WebViewImpl's compositor scheduling suppression out to WebWidgetClient
index 0d0a073..60ad540 100644 (file)
@@ -53,8 +53,6 @@ public:
     enum BrowserDataHint {
         BrowserDataHintNone,
         BrowserDataHintScreenshot,
-        BrowserDataHintAcceptJavaScriptDialog,
-        BrowserDataHintDismissJavaScriptDialog,
     };
 
     virtual ~WebDevToolsAgent() {}
index 91c7366..4bd668c 100644 (file)
@@ -83,8 +83,6 @@ namespace WebKit {
 
 namespace BrowserDataHintStringValues {
 static const char screenshot[] = "screenshot";
-static const char acceptJavaScriptDialog[] = "acceptJavaScriptDialog";
-static const char dismissJavaScriptDialog[] = "dismissJavaScriptDialog";
 }
 
 class ClientMessageLoopAdapter : public PageScriptDebugServer::ClientMessageLoop {
@@ -610,8 +608,7 @@ bool WebDevToolsAgentImpl::captureScreenshot(String* data)
 
 bool WebDevToolsAgentImpl::handleJavaScriptDialog(bool accept)
 {
-    // Operation is going to be performed in the browser process.
-    m_sendWithBrowserDataHint = accept ? BrowserDataHintAcceptJavaScriptDialog : BrowserDataHintDismissJavaScriptDialog;
+    // Operation was already performed in the browser process.
     return true;
 }
 
@@ -694,10 +691,6 @@ static String browserHintToString(WebDevToolsAgent::BrowserDataHint dataHint)
     switch (dataHint) {
     case WebDevToolsAgent::BrowserDataHintScreenshot:
         return BrowserDataHintStringValues::screenshot;
-    case WebDevToolsAgent::BrowserDataHintAcceptJavaScriptDialog:
-        return BrowserDataHintStringValues::acceptJavaScriptDialog;
-    case WebDevToolsAgent::BrowserDataHintDismissJavaScriptDialog:
-        return BrowserDataHintStringValues::dismissJavaScriptDialog;
     case WebDevToolsAgent::BrowserDataHintNone:
     default:
         ASSERT_NOT_REACHED();
@@ -709,10 +702,6 @@ static WebDevToolsAgent::BrowserDataHint browserHintFromString(const String& val
 {
     if (value == BrowserDataHintStringValues::screenshot)
         return WebDevToolsAgent::BrowserDataHintScreenshot;
-    if (value == BrowserDataHintStringValues::acceptJavaScriptDialog)
-        return WebDevToolsAgent::BrowserDataHintAcceptJavaScriptDialog;
-    if (value == BrowserDataHintStringValues::dismissJavaScriptDialog)
-        return WebDevToolsAgent::BrowserDataHintDismissJavaScriptDialog;
     ASSERT_NOT_REACHED();
     return WebDevToolsAgent::BrowserDataHintNone;
 }
@@ -868,10 +857,6 @@ WebString WebDevToolsAgent::patchWithBrowserData(const WebString& message, Brows
     case BrowserDataHintScreenshot:
         resultObject->setString("data", hintData);
         break;
-    case BrowserDataHintAcceptJavaScriptDialog:
-        break;
-    case BrowserDataHintDismissJavaScriptDialog:
-        break;
     case BrowserDataHintNone:
     default:
         ASSERT_NOT_REACHED();