[haiku-webkit-commits] r215 - webkit/trunk/WebKit/haiku/API

  • From: webkit@xxxxxxxxxxxxxxx
  • To: haiku-webkit-commits@xxxxxxxxxxxxx
  • Date: Thu, 25 Feb 2010 15:36:30 +0000

Author: stippi
Date: Thu Feb 25 14:16:55 2010
New Revision: 215
URL: http://mmlr.dyndns.org/changeset/215

Log:
Removed a bunch of now unimportant debugging output and added notes to some
events that I have better understanding of now.

Modified:
   webkit/trunk/WebKit/haiku/API/WebPage.cpp
   webkit/trunk/WebKit/haiku/API/WebWindow.cpp

Modified: webkit/trunk/WebKit/haiku/API/WebPage.cpp
==============================================================================
--- webkit/trunk/WebKit/haiku/API/WebPage.cpp   Thu Feb 25 14:16:06 2010        
(r214)
+++ webkit/trunk/WebKit/haiku/API/WebPage.cpp   Thu Feb 25 14:16:55 2010        
(r215)
@@ -200,7 +200,6 @@
 
 BWebPage::~BWebPage()
 {
-printf("~BWebPage()\n");
     delete m_mainFrame;
     delete m_page;
 }

Modified: webkit/trunk/WebKit/haiku/API/WebWindow.cpp
==============================================================================
--- webkit/trunk/WebKit/haiku/API/WebWindow.cpp Thu Feb 25 14:16:06 2010        
(r214)
+++ webkit/trunk/WebKit/haiku/API/WebWindow.cpp Thu Feb 25 14:16:55 2010        
(r215)
@@ -66,16 +66,21 @@
 {
     switch (message->what) {
     case LOAD_ONLOAD_HANDLE:
-        printf("LOAD_ONLOAD_HANDLE\n");
+        // NOTE: Supposedly this notification is sent when the so-called
+        // "onload" event has been passed and executed to scripts in the
+        // site.
         break;
     case LOAD_DL_COMPLETED:
-        printf("LOAD_DL_COMPLETED\n");
+        // NOTE: All loading has finished. We currently handle this via
+        // the progress notification. But it doesn't hurt to call the hook
+        // one more time.
+        LoadProgress(100, _WebViewForMessage(message));
         break;
     case LOAD_DOC_COMPLETED:
-        printf("LOAD_DOC_COMPLETED\n");
+        // NOTE: This events means the DOM document is ready.
         break;
     case JAVASCRIPT_WINDOW_OBJECT_CLEARED:
-        printf("JAVASCRIPT_WINDOW_OBJECT_CLEARED\n");
+        // NOTE: No idea what this event actually means.
         break;
     case NAVIGATION_REQUESTED: {
         BString url;

Other related posts:

  • » [haiku-webkit-commits] r215 - webkit/trunk/WebKit/haiku/API - webkit