Changeset 33029 in webkit

Timestamp:

May 10, 2008 3:01:34 PM (16 years ago)

Author:

adele@apple.com

Message:

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.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/PageGroup.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+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>
 

trunk/WebCore/page/PageGroup.cpp

@@ -61 +61 @@
 {
     ASSERT(page);
-    m_pages.add(page);
+    addPage(page);
 }