[Qt] QtWebKit will not compile with QT_ASCII_CAST_WARNINGS enabled
authorbenjamin.poulain@nokia.com <benjamin.poulain@nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Mar 2011 17:25:34 +0000 (17:25 +0000)
committerbenjamin.poulain@nokia.com <benjamin.poulain@nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Mar 2011 17:25:34 +0000 (17:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=57087

Reviewed by Andreas Kling.

Use explicit conversion for string to avoid depending on the default codec
installed by the user code.

* Platform/qt/SharedMemoryQt.cpp:
(WebKit::SharedMemory::create):
* Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:
(WebKit::initializeGTK):
* UIProcess/API/qt/tests/qgraphicswkview/tst_qgraphicswkview.cpp:
(tst_QGraphicsWKView::loadEmptyPage):
(tst_QGraphicsWKView::loadEmptyUrl):
* UIProcess/API/qt/tests/qwkpage/tst_qwkpage.cpp:
(tst_QWKPage::loadEmptyUrl):
* UIProcess/Launcher/qt/ProcessLauncherQt.cpp:
(WebKit::ProcessLauncher::launchProcess):
* WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:
(WebKit::getPluginDisplay):
* WebProcess/qt/WebProcessMainQt.cpp:
(WebKit::EnvHttpProxyFactory::initializeFromEnvironment):
(WebKit::WebProcessMainQt):

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

Source/WebKit2/ChangeLog
Source/WebKit2/Platform/qt/SharedMemoryQt.cpp
Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp
Source/WebKit2/UIProcess/API/qt/tests/qgraphicswkview/tst_qgraphicswkview.cpp
Source/WebKit2/UIProcess/API/qt/tests/qwkpage/tst_qwkpage.cpp
Source/WebKit2/UIProcess/Launcher/qt/ProcessLauncherQt.cpp
Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp
Source/WebKit2/WebProcess/qt/WebProcessMainQt.cpp

index b3f73b6..7bc958d 100644 (file)
@@ -1,3 +1,30 @@
+2011-03-25  Benjamin Poulain  <benjamin.poulain@nokia.com>
+
+        Reviewed by Andreas Kling.
+
+        [Qt] QtWebKit will not compile with QT_ASCII_CAST_WARNINGS enabled
+        https://bugs.webkit.org/show_bug.cgi?id=57087
+
+        Use explicit conversion for string to avoid depending on the default codec
+        installed by the user code.
+
+        * Platform/qt/SharedMemoryQt.cpp:
+        (WebKit::SharedMemory::create):
+        * Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:
+        (WebKit::initializeGTK):
+        * UIProcess/API/qt/tests/qgraphicswkview/tst_qgraphicswkview.cpp:
+        (tst_QGraphicsWKView::loadEmptyPage):
+        (tst_QGraphicsWKView::loadEmptyUrl):
+        * UIProcess/API/qt/tests/qwkpage/tst_qwkpage.cpp:
+        (tst_QWKPage::loadEmptyUrl):
+        * UIProcess/Launcher/qt/ProcessLauncherQt.cpp:
+        (WebKit::ProcessLauncher::launchProcess):
+        * WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:
+        (WebKit::getPluginDisplay):
+        * WebProcess/qt/WebProcessMainQt.cpp:
+        (WebKit::EnvHttpProxyFactory::initializeFromEnvironment):
+        (WebKit::WebProcessMainQt):
+
 2011-03-25  Brent Fulgham  <bfulgham@webkit.org>
 
         Correct project file missing $(WebKitVSPropsRedirectionDir)s
 2011-03-25  Brent Fulgham  <bfulgham@webkit.org>
 
         Correct project file missing $(WebKitVSPropsRedirectionDir)s
index 91af533..8d5d70f 100644 (file)
@@ -101,7 +101,7 @@ void SharedMemory::Handle::adoptFromAttachment(int fileDescriptor, size_t size)
 
 PassRefPtr<SharedMemory> SharedMemory::create(size_t size)
 {
 
 PassRefPtr<SharedMemory> SharedMemory::create(size_t size)
 {
-    QString tempName = QDir::temp().filePath("qwkshm.XXXXXX");
+    QString tempName = QDir::temp().filePath(QLatin1String("qwkshm.XXXXXX"));
     QByteArray tempNameCSTR = tempName.toLocal8Bit();
     char* tempNameC = tempNameCSTR.data();
 
     QByteArray tempNameCSTR = tempName.toLocal8Bit();
     char* tempNameC = tempNameCSTR.data();
 
index b5e3aad..76ecda7 100644 (file)
@@ -40,7 +40,7 @@ namespace WebKit {
 #if PLATFORM(QT)
 static void initializeGTK()
 {
 #if PLATFORM(QT)
 static void initializeGTK()
 {
-    QLibrary library("libgtk-x11-2.0.so.0");
+    QLibrary library(QLatin1String("libgtk-x11-2.0.so.0"));
     if (library.load()) {
         typedef void *(*gtk_init_check_ptr)(int*, char***);
         gtk_init_check_ptr gtkInitCheck = reinterpret_cast<gtk_init_check_ptr>(library.resolve("gtk_init_check"));
     if (library.load()) {
         typedef void *(*gtk_init_check_ptr)(int*, char***);
         gtk_init_check_ptr gtkInitCheck = reinterpret_cast<gtk_init_check_ptr>(library.resolve("gtk_init_check"));
index 655cadc..8cb7263 100644 (file)
@@ -84,7 +84,7 @@ void tst_QGraphicsWKView::loadEmptyPage()
 {
     m_view->show();
 
 {
     m_view->show();
 
-    m_view->m_webView-> load(QUrl::fromLocalFile(TESTS_SOURCE_DIR "/html/basic_page.html"));
+    m_view->m_webView-> load(QUrl::fromLocalFile(QLatin1String(TESTS_SOURCE_DIR "/html/basic_page.html")));
     QVERIFY(waitForSignal(m_view->m_webView, SIGNAL(loadFinished(bool))));
 }
 
     QVERIFY(waitForSignal(m_view->m_webView, SIGNAL(loadFinished(bool))));
 }
 
@@ -95,7 +95,7 @@ void tst_QGraphicsWKView::loadEmptyUrl()
     m_view->m_webView->load(QUrl());
     QVERIFY(!waitForSignal(m_view->m_webView->page(), SIGNAL(engineConnectionChanged(bool)), 50));
 
     m_view->m_webView->load(QUrl());
     QVERIFY(!waitForSignal(m_view->m_webView->page(), SIGNAL(engineConnectionChanged(bool)), 50));
 
-    m_view->m_webView->load(QUrl(""));
+    m_view->m_webView->load(QUrl(QLatin1String("")));
     QVERIFY(!waitForSignal(m_view->m_webView->page(), SIGNAL(engineConnectionChanged(bool)), 50));
 }
 
     QVERIFY(!waitForSignal(m_view->m_webView->page(), SIGNAL(engineConnectionChanged(bool)), 50));
 }
 
index 3b64d05..39fcac2 100644 (file)
@@ -51,7 +51,7 @@ void tst_QWKPage::cleanup()
 void tst_QWKPage::loadEmptyUrl()
 {
     m_page->load(QUrl());
 void tst_QWKPage::loadEmptyUrl()
 {
     m_page->load(QUrl());
-    m_page->load(QUrl(""));
+    m_page->load(QUrl(QLatin1String("")));
 }
 
 QTEST_MAIN(tst_QWKPage)
 }
 
 QTEST_MAIN(tst_QWKPage)
index dbe3f10..cae5bdf 100644 (file)
@@ -85,12 +85,12 @@ void QtWebProcess::setupChildProcess()
 
 void ProcessLauncher::launchProcess()
 {
 
 void ProcessLauncher::launchProcess()
 {
-    QString applicationPath = "%1 %2";
+    QString applicationPath = QLatin1String("%1 %2");
 
 
-    if (QFile::exists(QCoreApplication::applicationDirPath() + "/QtWebProcess")) {
-        applicationPath = applicationPath.arg(QCoreApplication::applicationDirPath() + "/QtWebProcess");
+    if (QFile::exists(QCoreApplication::applicationDirPath() + QLatin1String("/QtWebProcess"))) {
+        applicationPath = applicationPath.arg(QCoreApplication::applicationDirPath() + QLatin1String("/QtWebProcess"));
     } else {
     } else {
-        applicationPath = applicationPath.arg("QtWebProcess");
+        applicationPath = applicationPath.arg(QLatin1String("QtWebProcess"));
     }
 
     int sockets[2];
     }
 
     int sockets[2];
index 8c1fef6..27e694b 100644 (file)
@@ -51,7 +51,7 @@ static Display *getPluginDisplay()
     // The code below has the same effect as this one:
     // Display *gdkDisplay = gdk_x11_display_get_xdisplay(gdk_display_get_default());
 
     // The code below has the same effect as this one:
     // Display *gdkDisplay = gdk_x11_display_get_xdisplay(gdk_display_get_default());
 
-    QLibrary library("libgdk-x11-2.0", 0);
+    QLibrary library(QLatin1String("libgdk-x11-2.0"), 0);
     if (!library.load())
         return 0;
 
     if (!library.load())
         return 0;
 
index fadf545..6ff10ba 100644 (file)
@@ -82,7 +82,7 @@ bool EnvHttpProxyFactory::initializeFromEnvironment()
 {
     bool wasSetByEnvironment = false;
 
 {
     bool wasSetByEnvironment = false;
 
-    QUrl proxyUrl = QUrl::fromUserInput(qgetenv("http_proxy"));
+    QUrl proxyUrl = QUrl::fromUserInput(QString::fromLocal8Bit(qgetenv("http_proxy")));
     if (proxyUrl.isValid() && !proxyUrl.host().isEmpty()) {
         int proxyPort = (proxyUrl.port() > 0) ? proxyUrl.port() : 8080;
         m_httpProxy << QNetworkProxy(QNetworkProxy::HttpProxy, proxyUrl.host(), proxyPort);
     if (proxyUrl.isValid() && !proxyUrl.host().isEmpty()) {
         int proxyPort = (proxyUrl.port() > 0) ? proxyUrl.port() : 8080;
         m_httpProxy << QNetworkProxy(QNetworkProxy::HttpProxy, proxyUrl.host(), proxyPort);
@@ -90,7 +90,7 @@ bool EnvHttpProxyFactory::initializeFromEnvironment()
     } else
         m_httpProxy << QNetworkProxy::NoProxy;
 
     } else
         m_httpProxy << QNetworkProxy::NoProxy;
 
-    proxyUrl = QUrl::fromUserInput(qgetenv("https_proxy"));
+    proxyUrl = QUrl::fromUserInput(QString::fromLocal8Bit(qgetenv("https_proxy")));
     if (proxyUrl.isValid() && !proxyUrl.host().isEmpty()) {
         int proxyPort = (proxyUrl.port() > 0) ? proxyUrl.port() : 8080;
         m_httpsProxy << QNetworkProxy(QNetworkProxy::HttpProxy, proxyUrl.host(), proxyPort);
     if (proxyUrl.isValid() && !proxyUrl.host().isEmpty()) {
         int proxyPort = (proxyUrl.port() > 0) ? proxyUrl.port() : 8080;
         m_httpsProxy << QNetworkProxy(QNetworkProxy::HttpProxy, proxyUrl.host(), proxyPort);
@@ -132,7 +132,7 @@ static void initializeProxy()
 
 Q_DECL_EXPORT int WebProcessMainQt(int argc, char** argv)
 {
 
 Q_DECL_EXPORT int WebProcessMainQt(int argc, char** argv)
 {
-    QApplication::setGraphicsSystem("raster");
+    QApplication::setGraphicsSystem(QLatin1String("raster"));
     QApplication* app = new QApplication(argc, argv);
 #ifndef NDEBUG
     if (!qgetenv("WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH").isEmpty()) {
     QApplication* app = new QApplication(argc, argv);
 #ifndef NDEBUG
     if (!qgetenv("WEBKIT2_PAUSE_WEB_PROCESS_ON_LAUNCH").isEmpty()) {