more error handling changes.

[apitrace] / gui / apitracecall.cpp
diff --git a/gui/apitracecall.cpp b/gui/apitracecall.cpp

index bac3d73762ce1575902e3f4ce351045c3a6f0aa9..2d41e25a4425f972b56d6f0daafabd8c694d5863 100644 (file)
--- a/gui/apitracecall.cpp
+++ b/gui/apitracecall.cpp
@@ -30,14 +30,16 @@ const char * const styleSheet =
      "    color: #0000ff;\n"
      "}\n"
      ".error {\n"
+    "    border: 1px solid rgb(255,0,0);\n"
      "    margin: 10px;\n"
-    "    padding: 0;\n"
-    "    color: red;\n"
-    "    background: #6fb2e5;\n"
-    "    box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
-    "    -o-box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
-    "    -webkit-box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
-    "    -moz-box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
+    "    padding: 1;\n"
+    "    border-radius: 4px;\n"
+    // also looks great but qtwebkit doesn't support it
+    //"    background: #6fb2e5;\n"
+    //"    box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
+    //"    -o-box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
+    //"    -webkit-box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
+    //"    -moz-box-shadow: 0 1px 5px #0061aa, inset 0 10px 20px #b6f9ff;\n"
      "}\n";
  
  ApiPointer::ApiPointer(unsigned long long val)
@@ -387,6 +389,14 @@ QString ApiTraceCall::toHtml() const
      }
      m_richText += QLatin1String("</div>");
  
+    if (hasError()) {
+        QString errorStr =
+            QString::fromLatin1(
+                "<div class=\"error\">%1</div>")
+            .arg(m_error);
+        m_richText += errorStr;
+    }
+
      m_richText =
          QString::fromLatin1(
              "<html><head><style type=\"text/css\" media=\"all\">"
@@ -814,6 +824,7 @@ void ApiTraceCall::setError(const QString &msg)
      if (m_error != msg) {
          ApiTrace *trace = parentTrace();
          m_error = msg;
+        m_richText = QString();
          if (trace)
              trace->callError(this);
      }