From: adele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Sat, 10 May 2008 22:01:34 +0000 (+0000)
Subject: 2008-05-10  Adele Peterson  <adele@apple.com>
X-Git-Url: http://git.webkit.org/?p=WebKit-https.git;a=commitdiff_plain;h=b26e0928e9644d3624e9835d17ba8a4e20422a7c

2008-05-10  Adele Peterson  <adele@apple.com>

        Reviewed by Tim Hatcher.

        Fix crash for WebKit clients that don't set a group name for the WebView/Page.

        * page/PageGroup.cpp: (WebCore::PageGroup::PageGroup): Instead of adding the Page directly to the group,
          call the addPage method, since it does extra work initializing m_localStorage.



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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 6871686..6ffa6b4 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,12 @@
+2008-05-10  Adele Peterson  <adele@apple.com>
+
+        Reviewed by Tim Hatcher.
+
+        Fix crash for WebKit clients that don't set a group name for the WebView/Page.
+
+        * page/PageGroup.cpp: (WebCore::PageGroup::PageGroup): Instead of adding the Page directly to the group,
+          call the addPage method, since it does extra work initializing m_localStorage.
+
 2008-05-09  Sam Weinig  <sam@webkit.org>
 
         Reviewed by Mark Rowe.
diff --git a/WebCore/page/PageGroup.cpp b/WebCore/page/PageGroup.cpp
index 9f52004..b8af0cd 100644
--- a/WebCore/page/PageGroup.cpp
+++ b/WebCore/page/PageGroup.cpp
@@ -60,7 +60,7 @@ PageGroup::PageGroup(Page* page)
     , m_identifier(getUniqueIdentifier())
 {
     ASSERT(page);
-    m_pages.add(page);
+    addPage(page);
 }
 
 typedef HashMap<String, PageGroup*> PageGroupMap;