From c6d4ff531fb91e92c03525443cf914cd6fe73e8d Mon Sep 17 00:00:00 2001
From: Friedemann Kleint <Friedemann.Kleint@nokia.com>
Date: Fri, 4 Feb 2011 09:11:24 +0100
Subject: [PATCH] Debugger: Fix breakpoint dialog.

Update model once a breakpoint is added, set initial
state of dialog.
---
 src/plugins/debugger/breakhandler.cpp | 6 ++++++
 src/plugins/debugger/breakwindow.cpp  | 6 ++++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/plugins/debugger/breakhandler.cpp b/src/plugins/debugger/breakhandler.cpp
index 078f86a634e..d310a0fc360 100644
--- a/src/plugins/debugger/breakhandler.cpp
+++ b/src/plugins/debugger/breakhandler.cpp
@@ -832,9 +832,15 @@ void BreakHandler::appendBreakpoint(const BreakpointParameters &data)
     item.data = data;
     item.response.fileName = data.fileName;
     item.response.lineNumber = data.lineNumber;
+
+    const int row = m_storage.size();
+    beginInsertRows(QModelIndex(), row, row);
     m_storage.insert(id, item);
+    endInsertRows();
+
     updateMarker(id);
     scheduleSynchronization();
+
 }
 
 void BreakHandler::saveSessionData()
diff --git a/src/plugins/debugger/breakwindow.cpp b/src/plugins/debugger/breakwindow.cpp
index 41e41595c7e..7dfa5c2a81b 100644
--- a/src/plugins/debugger/breakwindow.cpp
+++ b/src/plugins/debugger/breakwindow.cpp
@@ -100,10 +100,11 @@ private:
     Ui::BreakpointDialog m_ui;
     BreakpointParameters m_savedParameters;
     BreakpointType m_previousType;
+    bool m_firstTypeChange;
 };
 
 BreakpointDialog::BreakpointDialog(QWidget *parent)
-  : QDialog(parent), m_previousType(UnknownType)
+    : QDialog(parent), m_previousType(UnknownType), m_firstTypeChange(true)
 {
     // Match BreakpointType (omitting unknown type).
     m_ui.setupUi(this);
@@ -128,9 +129,10 @@ BreakpointDialog::BreakpointDialog(QWidget *parent)
 void BreakpointDialog::setType(BreakpointType type)
 {
     const int comboIndex = type - 1; // Skip UnknownType.
-    if (comboIndex != m_ui.comboBoxType->currentIndex()) {
+    if (comboIndex != m_ui.comboBoxType->currentIndex() || m_firstTypeChange) {
         m_ui.comboBoxType->setCurrentIndex(comboIndex);
         typeChanged(comboIndex);
+        m_firstTypeChange = false;
     }
 }