2008-05-10 Adele Peterson <adele@apple.com>
authoradele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 10 May 2008 22:01:34 +0000 (22:01 +0000)
committeradele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 10 May 2008 22:01:34 +0000 (22:01 +0000)
        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

WebCore/ChangeLog
WebCore/page/PageGroup.cpp

index 6871686..6ffa6b4 100644 (file)
@@ -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.
index 9f52004..b8af0cd 100644 (file)
@@ -60,7 +60,7 @@ PageGroup::PageGroup(Page* page)
     , m_identifier(getUniqueIdentifier())
 {
     ASSERT(page);
-    m_pages.add(page);
+    addPage(page);
 }
 
 typedef HashMap<String, PageGroup*> PageGroupMap;