2011-05-24 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 May 2011 03:39:21 +0000 (03:39 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 May 2011 03:39:21 +0000 (03:39 +0000)
        Reviewed by Eric Seidel.

        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
        https://bugs.webkit.org/show_bug.cgi?id=61395

        All the callers of the API are wrong.  They should call the more modern
        static member function of ScriptController instead.  I'll remove the
        FrameLoader function in a future patch.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::processingUserGesture):
        * rendering/RenderFileUploadControl.cpp:
        (WebCore::RenderFileUploadControl::click):
2011-05-24  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
        https://bugs.webkit.org/show_bug.cgi?id=61395

        * src/FrameLoaderClientImpl.cpp:
        (WebKit::FrameLoaderClientImpl::dispatchDidNavigateWithinPage):
        * src/WebFrameImpl.cpp:
        (WebKit::WebFrameImpl::isProcessingUserGesture):
2011-05-24  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
        https://bugs.webkit.org/show_bug.cgi?id=61395

        * WebView/WebView.mm:
        (-[WebView _isProcessingUserGesture]):
2011-05-24  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
        https://bugs.webkit.org/show_bug.cgi?id=61395

        * WebCoreSupport/FrameLoaderClientQt.cpp:
        (drtPrintFrameUserGestureStatus):

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

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLMediaElement.cpp
Source/WebCore/rendering/RenderFileUploadControl.cpp
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/FrameLoaderClientImpl.cpp
Source/WebKit/chromium/src/WebFrameImpl.cpp
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/WebView/WebView.mm
Source/WebKit/qt/ChangeLog
Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp

index 2af108c..5eb52e7 100644 (file)
@@ -1,3 +1,19 @@
+2011-05-24  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
+        https://bugs.webkit.org/show_bug.cgi?id=61395
+
+        All the callers of the API are wrong.  They should call the more modern
+        static member function of ScriptController instead.  I'll remove the
+        FrameLoader function in a future patch.
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::processingUserGesture):
+        * rendering/RenderFileUploadControl.cpp:
+        (WebCore::RenderFileUploadControl::click):
+
 2011-05-24  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
 
         Unreviewed. Fix style error.
index a6d2def..a44f9a7 100644 (file)
@@ -2402,11 +2402,12 @@ void HTMLMediaElement::defaultEventHandler(Event* event)
 
 bool HTMLMediaElement::processingUserGesture() const
 {
+    // FIXME: We should call ScriptController::processingUserGesture() so
+    // we know what to do without a Frame.
     Frame* frame = document()->frame();
-    FrameLoader* loader = frame ? frame->loader() : 0;
-
-    // return 'true' for safety if we don't know the answer 
-    return loader ? loader->isProcessingUserGesture() : true;
+    if (!frame)
+        return true;
+    return frame->loader()->isProcessingUserGesture();
 }
 
 #if ENABLE(PLUGIN_PROXY_FOR_VIDEO)
index 47d76bd..712e8f5 100644 (file)
@@ -130,7 +130,7 @@ void RenderFileUploadControl::chooseIconForFiles(FileChooser* chooser, const Vec
 
 void RenderFileUploadControl::click()
 {
-    // Requires a user gesture to open the file dialog.
+    // FIXME: We should call ScriptController::processingUserGesture().
     if (!frame() || !frame()->loader()->isProcessingUserGesture())
         return;
     if (Chrome* chromePointer = chrome())
index 68b8d86..d02ae7e 100644 (file)
@@ -1,3 +1,15 @@
+2011-05-24  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
+        https://bugs.webkit.org/show_bug.cgi?id=61395
+
+        * src/FrameLoaderClientImpl.cpp:
+        (WebKit::FrameLoaderClientImpl::dispatchDidNavigateWithinPage):
+        * src/WebFrameImpl.cpp:
+        (WebKit::WebFrameImpl::isProcessingUserGesture):
+
 2011-05-24  Greg Simon  <gregsimon@chromium.org>
 
         Reviewed by Dimitri Glazkov.
index f924862..6bd0cfc 100644 (file)
@@ -634,6 +634,9 @@ void FrameLoaderClientImpl::dispatchDidNavigateWithinPage()
             // insufficient to catch and properly flag these transitions. Once a
             // proper fix for this bug is identified and applied the following
             // block may no longer be required.
+            //
+            // FIXME: Why do we call isProcessingUserGesture here but none of
+            // the other ports do?
             bool wasClientRedirect =
                 (url == m_expectedClientRedirectDest && chainEnd == m_expectedClientRedirectSrc)
                 || !m_webFrame->isProcessingUserGesture();
index 4eb3c78..418d500 100644 (file)
@@ -1099,6 +1099,7 @@ unsigned WebFrameImpl::unloadListenerCount() const
 
 bool WebFrameImpl::isProcessingUserGesture() const
 {
+    // FIXME: We should call ScriptController::processingUserGesture().
     return frame()->loader()->isProcessingUserGesture();
 }
 
index 14067a5..53aee00 100644 (file)
@@ -1,3 +1,13 @@
+2011-05-24  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
+        https://bugs.webkit.org/show_bug.cgi?id=61395
+
+        * WebView/WebView.mm:
+        (-[WebView _isProcessingUserGesture]):
+
 2011-05-24  Michael Saboff  <msaboff@apple.com>
 
         Reviewed by Geoffrey Garen.
index 196650a..7f8ebab 100644 (file)
@@ -1238,6 +1238,7 @@ static bool fastDocumentTeardownEnabled()
 - (BOOL)_isProcessingUserGesture
 {
     WebFrame *frame = [self mainFrame];
+    // FIXME: We should call ScriptController::processingUserGesture().
     return core(frame)->loader()->isProcessingUserGesture();
 }
 
index d97404d..995e575 100644 (file)
@@ -1,3 +1,13 @@
+2011-05-24  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Add FIXME comments about removing FrameLoader::isProcessingUserGesture
+        https://bugs.webkit.org/show_bug.cgi?id=61395
+
+        * WebCoreSupport/FrameLoaderClientQt.cpp:
+        (drtPrintFrameUserGestureStatus):
+
 2011-05-23  Luiz Agostini  <luiz.agostini@openbossa.org>
 
         Reviewed by Andreas Kling.
index 56be324..71d21ad 100644 (file)
@@ -113,6 +113,7 @@ static QString drtDescriptionSuitableForTestResult(WebCore::Frame* webCoreFrame)
 
 static QString drtPrintFrameUserGestureStatus(WebCore::Frame* frame)
 {
+    // FIXME: We should call ScriptController::processingUserGesture().
     if (frame->loader()->isProcessingUserGesture())
         return QString::fromLatin1("Frame with user gesture \"%1\"").arg(QLatin1String("true"));
     return QString::fromLatin1("Frame with user gesture \"%1\"").arg(QLatin1String("false"));