Changeset 89870 in webkit

Timestamp:

Jun 27, 2011 3:31:36 PM (13 years ago)

Author:

adamk@chromium.org

Message:

2011-06-27 Adam Klein <​adamk@chromium.org>

Reviewed by Adam Barth.

Fix filesystem-no-callback-null-ptr-crash.html test to reference proper file api methods
​https://bugs.webkit.org/show_bug.cgi?id=63486

• fast/filesystem/filesystem-no-callback-null-ptr-crash.html:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• fast/filesystem/filesystem-no-callback-null-ptr-crash.html (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2011-06-27  Adam Klein  <adamk@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        Fix filesystem-no-callback-null-ptr-crash.html test to reference proper file api methods
+        https://bugs.webkit.org/show_bug.cgi?id=63486
+
+        * fast/filesystem/filesystem-no-callback-null-ptr-crash.html:
+
 2011-06-27  Jay Civelli  <jcivelli@chromium.org>
 

trunk/LayoutTests/fast/filesystem/filesystem-no-callback-null-ptr-crash.html

@@ -7 +7 @@
   if (window.layoutTestController)
       layoutTestController.dumpAsText();
-  // These two calls so not cause a NULL pointer crash:
-  try { window.requestFileSystem(10); } catch (e) {}
-  try { window.resolveLocalFileSystemURI(); } catch (e) {}
-  document.getElementById('log').innerHTML = "PASS";
+  if (window.webkitRequestFileSystem) {
+      // These two calls should not cause a NULL pointer crash:
+      webkitRequestFileSystem(TEMPORARY, 100);
+      webkitResolveLocalFileSystemURL('');
+      document.getElementById('log').innerHTML = "PASS";
+  }
 </script>