Title: [103238] trunk/Source/WebKit2
Revision
103238
Author
caio.olive...@openbossa.org
Date
2011-12-19 07:34:01 -0800 (Mon, 19 Dec 2011)

Log Message

[Qt] [WK2] Fix build error due to unused variables
https://bugs.webkit.org/show_bug.cgi?id=74862

Reviewed by Kenneth Rohde Christiansen.

* UIProcess/qt/QtWebPageEventHandler.cpp:
(QtWebPageEventHandler::inputMethodEvent):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (103237 => 103238)


--- trunk/Source/WebKit2/ChangeLog	2011-12-19 15:29:51 UTC (rev 103237)
+++ trunk/Source/WebKit2/ChangeLog	2011-12-19 15:34:01 UTC (rev 103238)
@@ -1,3 +1,13 @@
+2011-12-19  Caio Marcelo de Oliveira Filho  <caio.olive...@openbossa.org>
+
+        [Qt] [WK2] Fix build error due to unused variables
+        https://bugs.webkit.org/show_bug.cgi?id=74862
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        * UIProcess/qt/QtWebPageEventHandler.cpp:
+        (QtWebPageEventHandler::inputMethodEvent):
+
 2011-12-19  Kenneth Rohde Christiansen  <kenn...@webkit.org>
 
         [Qt] Add methods for setting, cancelling and confirming the composition

Modified: trunk/Source/WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp (103237 => 103238)


--- trunk/Source/WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp	2011-12-19 15:29:51 UTC (rev 103237)
+++ trunk/Source/WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp	2011-12-19 15:34:01 UTC (rev 103238)
@@ -361,16 +361,15 @@
                 selectionStart = attr.start;
                 selectionLength = attr.length;
 
-                ASSERT(selectionStart >= 0);
-                ASSERT(selectionLength >= 0);
+                ASSERT_UNUSED(selectionStart, selectionStart >= 0);
+                ASSERT_UNUSED(selectionLength, selectionLength >= 0);
                 break;
             }
         }
 
         // FIXME: Confirm the composition here.
     } else {
-        ASSERT(cursorPositionWithinComposition >= 0);
-        ASSERT(replacementStart >= 0);
+        ASSERT_UNUSED(cursorPositionWithinComposition, cursorPositionWithinComposition >= 0);
 
         // FIXME: Set the composition here.
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to