another segfault fix

[wxWidgets.git] / src / html / htmlwin.cpp
diff --git a/src/html/htmlwin.cpp b/src/html/htmlwin.cpp

index c029c706fc9fa94469069fd7aae0ab334bd42192..6cf959cf5f1bcb788e129e68d1fa204ebb2e5ecd 100644 (file)
--- a/src/html/htmlwin.cpp
+++ b/src/html/htmlwin.cpp
@@ -72,8 +72,7 @@ WX_DEFINE_LIST(wxHtmlProcessorList);
  //-----------------------------------------------------------------------------
  
  
-wxHtmlWindow::wxHtmlWindow(wxWindow *parent, wxWindowID id, const wxPoint& pos, const wxSize& size,
-                long style, const wxString& name) : wxScrolledWindow(parent, id, pos, size, style | wxVSCROLL | wxHSCROLL, name)
+void wxHtmlWindow::Init()
  {
      m_tmpMouseMoved = FALSE;
      m_tmpLastLink = NULL;
@@ -87,17 +86,28 @@ wxHtmlWindow::wxHtmlWindow(wxWindow *parent, wxWindowID id, const wxPoint& pos,
      m_Cell = NULL;
      m_Parser = new wxHtmlWinParser(this);
      m_Parser->SetFS(m_FS);
-    SetBorders(10);
      m_HistoryPos = -1;
      m_HistoryOn = TRUE;
      m_History = new wxHtmlHistoryArray;
      m_Processors = NULL;
+    m_Style = 0;
+    SetBorders(10);
+}
+
+bool wxHtmlWindow::Create(wxWindow *parent, wxWindowID id, 
+                          const wxPoint& pos, const wxSize& size,
+                          long style, const wxString& name) 
+{
+    if (!wxScrolledWindow::Create(parent, id, pos, size, 
+                                  style | wxVSCROLL | wxHSCROLL, name))
+        return FALSE;
+
      m_Style = style;
      SetPage(wxT("<html><body></body></html>"));
+    return TRUE;
  }
  
  
-
  wxHtmlWindow::~wxHtmlWindow()
  {
      HistoryClear();
@@ -197,6 +207,10 @@ bool wxHtmlWindow::SetPage(const wxString& source)
      return TRUE;
  }
  
+bool wxHtmlWindow::AppendToPage(const wxString& source)
+{
+    return SetPage(*(GetParser()->GetSource()) + source);
+}
  
  bool wxHtmlWindow::LoadPage(const wxString& location)
  {
@@ -614,7 +628,7 @@ void wxHtmlWindow::OnLinkClicked(const wxHtmlLinkInfo& link)
  void wxHtmlWindow::OnCellClicked(wxHtmlCell *cell,
                                   wxCoord x, wxCoord y,
                                   const wxMouseEvent& event)
-{ 
+{
      wxCHECK_RET( cell, _T("can't be called with NULL cell") );
  
      cell->OnMouseClick(this, x, y, event);
@@ -628,23 +642,18 @@ void wxHtmlWindow::OnCellMouseHover(wxHtmlCell * WXUNUSED(cell),
  
  void wxHtmlWindow::OnDraw(wxDC& dc)
  {
-    int x, y;
-    wxRegionIterator upd(GetUpdateRegion()); // get the update rect list
-    int v_y, v_h;
+    if (m_tmpCanDrawLocks > 0 || m_Cell == NULL) return;
  
-    if (m_tmpCanDrawLocks > 0) return;
+    int x, y;
+    wxRect rect = GetUpdateRegion().GetBox();
  
      dc.SetMapMode(wxMM_TEXT);
      dc.SetBackgroundMode(wxTRANSPARENT);
      GetViewStart(&x, &y);
  
-    while (upd)
-    {
-        v_y = upd.GetY();
-        v_h = upd.GetH();
-        if (m_Cell) m_Cell->Draw(dc, 0, 0, y * wxHTML_SCROLL_STEP + v_y, y * wxHTML_SCROLL_STEP + v_h + v_y);
-        upd++;
-    }
+    m_Cell->Draw(dc, 0, 0, 
+                 y * wxHTML_SCROLL_STEP + rect.GetTop(), 
+                 y * wxHTML_SCROLL_STEP + rect.GetBottom());
  }