Title: [123850] trunk/Source/WebCore
Revision
123850
Author
[email protected]
Date
2012-07-27 01:34:14 -0700 (Fri, 27 Jul 2012)

Log Message

Page object should ascertain EditorClient to be non-null
https://bugs.webkit.org/show_bug.cgi?id=92458

Patch by Vivek Galatage <[email protected]> on 2012-07-27
Reviewed by Ryosuke Niwa.

The EditorClient should always be available in Page object. Added the assertion to ascertain this.

No new tests. Added assertion to make sure EditorClient always exist.

* page/Page.cpp:
(WebCore::Page::Page):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (123849 => 123850)


--- trunk/Source/WebCore/ChangeLog	2012-07-27 08:28:51 UTC (rev 123849)
+++ trunk/Source/WebCore/ChangeLog	2012-07-27 08:34:14 UTC (rev 123850)
@@ -1,3 +1,17 @@
+2012-07-27  Vivek Galatage  <[email protected]>
+
+        Page object should ascertain EditorClient to be non-null
+        https://bugs.webkit.org/show_bug.cgi?id=92458
+
+        Reviewed by Ryosuke Niwa.
+
+        The EditorClient should always be available in Page object. Added the assertion to ascertain this.
+
+        No new tests. Added assertion to make sure EditorClient always exist.
+
+        * page/Page.cpp:
+        (WebCore::Page::Page):
+
 2012-07-27  Huang Dongsung  <[email protected]>
 
         Gather the duplicated timer code into CachedResource.

Modified: trunk/Source/WebCore/page/Page.cpp (123849 => 123850)


--- trunk/Source/WebCore/page/Page.cpp	2012-07-27 08:28:51 UTC (rev 123849)
+++ trunk/Source/WebCore/page/Page.cpp	2012-07-27 08:34:14 UTC (rev 123850)
@@ -167,6 +167,8 @@
     , m_alternativeTextClient(pageClients.alternativeTextClient)
     , m_scriptedAnimationsSuspended(false)
 {
+    ASSERT(m_editorClient);
+
     if (!allPages) {
         allPages = new HashSet<Page*>;
         
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to