Changeset 95526 in webkit

Timestamp:

Sep 20, 2011 4:36:15 AM (13 years ago)

Author:

vestbo@webkit.org

Message:

[Qt] Don't use QFont before initializing QApplication

Could potentially result in crash.

Reviewed by Simon Hausmann.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/qt/main.cpp (modified) (2 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2011-09-20  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
+
+        [Qt] Don't use QFont before initializing QApplication
+
+        Could potentially result in crash.
+
+        Reviewed by Simon Hausmann.
+
+        * DumpRenderTree/qt/main.cpp:
+        (main):
+
 2011-09-19  Adam Barth  <abarth@webkit.org>
 

trunk/Tools/DumpRenderTree/qt/main.cpp

@@ -145 +145 @@
     QApplication::setStyle(new QWindowsStyle);
 
+    QApplication app(argc, argv);
+
     QFont f("Sans Serif");
     f.setPointSize(9);
@@ -151 +153 @@
     QApplication::setFont(f);
 
-    QApplication app(argc, argv);
 #ifdef Q_WS_X11
     QX11Info::setAppDpiY(0, 96);