Title: [92315] trunk
Revision
92315
Author
commit-qu...@webkit.org
Date
2011-08-03 14:58:44 -0700 (Wed, 03 Aug 2011)

Log Message

Remove LegacyDefaultOptionalArguments flag from navigator IDL files
https://bugs.webkit.org/show_bug.cgi?id=65370

Patch by Mark Pilgrim <pilg...@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

Source/WebCore:

* page/Navigator.idl:
* page/NavigatorUserMediaErrorCallback.idl:
* page/NavigatorUserMediaSuccessCallback.idl:

LayoutTests:

* fast/dom/navigator-detached-no-crash-expected.txt:
* platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (92314 => 92315)


--- trunk/LayoutTests/ChangeLog	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/LayoutTests/ChangeLog	2011-08-03 21:58:44 UTC (rev 92315)
@@ -1,3 +1,13 @@
+2011-08-03  Mark Pilgrim  <pilg...@chromium.org>
+
+        Remove LegacyDefaultOptionalArguments flag from navigator IDL files
+        https://bugs.webkit.org/show_bug.cgi?id=65370
+
+        Reviewed by Adam Barth.
+
+        * fast/dom/navigator-detached-no-crash-expected.txt:
+        * platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt:
+
 2011-08-03  Zhenyao Mo  <z...@google.com>
 
         Unreviewed, fix an error in test_expectations.txt.

Modified: trunk/LayoutTests/fast/dom/navigator-detached-no-crash-expected.txt (92314 => 92315)


--- trunk/LayoutTests/fast/dom/navigator-detached-no-crash-expected.txt	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/LayoutTests/fast/dom/navigator-detached-no-crash-expected.txt	2011-08-03 21:58:44 UTC (rev 92315)
@@ -13,6 +13,7 @@
 navigator.plugins is OK
 navigator.product is OK
 navigator.productSub is OK
+navigator.registerProtocolHandler() threw err TypeError: Not enough arguments
 navigator.userAgent is OK
 navigator.vendor is OK
 navigator.vendorSub is OK
@@ -29,6 +30,7 @@
 navigator.plugins is OK
 navigator.product is OK
 navigator.productSub is OK
+navigator.registerProtocolHandler() threw err TypeError: Not enough arguments
 navigator.userAgent is OK
 navigator.vendor is OK
 navigator.vendorSub is OK

Modified: trunk/LayoutTests/platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt (92314 => 92315)


--- trunk/LayoutTests/platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/LayoutTests/platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt	2011-08-03 21:58:44 UTC (rev 92315)
@@ -13,7 +13,7 @@
 navigator.plugins is OK
 navigator.product is OK
 navigator.productSub is OK
-navigator.registerProtocolHandler() is OK
+navigator.registerProtocolHandler() threw err TypeError: Not enough arguments
 navigator.userAgent is OK
 navigator.vendor is OK
 navigator.vendorSub is OK
@@ -30,7 +30,7 @@
 navigator.plugins is OK
 navigator.product is OK
 navigator.productSub is OK
-navigator.registerProtocolHandler() is OK
+navigator.registerProtocolHandler() threw err TypeError: Not enough arguments
 navigator.userAgent is OK
 navigator.vendor is OK
 navigator.vendorSub is OK

Modified: trunk/Source/WebCore/ChangeLog (92314 => 92315)


--- trunk/Source/WebCore/ChangeLog	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/Source/WebCore/ChangeLog	2011-08-03 21:58:44 UTC (rev 92315)
@@ -1,3 +1,14 @@
+2011-08-03  Mark Pilgrim  <pilg...@chromium.org>
+
+        Remove LegacyDefaultOptionalArguments flag from navigator IDL files
+        https://bugs.webkit.org/show_bug.cgi?id=65370
+
+        Reviewed by Adam Barth.
+
+        * page/Navigator.idl:
+        * page/NavigatorUserMediaErrorCallback.idl:
+        * page/NavigatorUserMediaSuccessCallback.idl:
+
 2011-08-03  Anders Carlsson  <ander...@apple.com>
 
         webarchive/loading/missing-data.html is failing on Mac

Modified: trunk/Source/WebCore/page/Navigator.idl (92314 => 92315)


--- trunk/Source/WebCore/page/Navigator.idl	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/Source/WebCore/page/Navigator.idl	2011-08-03 21:58:44 UTC (rev 92315)
@@ -20,7 +20,6 @@
 module window {
 
     interface [
-        LegacyDefaultOptionalArguments,
         GenerateIsReachable=ImplFrame,
         OmitConstructor
     ] Navigator {

Modified: trunk/Source/WebCore/page/NavigatorUserMediaErrorCallback.idl (92314 => 92315)


--- trunk/Source/WebCore/page/NavigatorUserMediaErrorCallback.idl	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/Source/WebCore/page/NavigatorUserMediaErrorCallback.idl	2011-08-03 21:58:44 UTC (rev 92315)
@@ -24,7 +24,6 @@
 
 module window {
     interface [
-        LegacyDefaultOptionalArguments,
         Conditional=MEDIA_STREAM,
         Callback=FunctionOnly
     ] NavigatorUserMediaErrorCallback {

Modified: trunk/Source/WebCore/page/NavigatorUserMediaSuccessCallback.idl (92314 => 92315)


--- trunk/Source/WebCore/page/NavigatorUserMediaSuccessCallback.idl	2011-08-03 21:53:16 UTC (rev 92314)
+++ trunk/Source/WebCore/page/NavigatorUserMediaSuccessCallback.idl	2011-08-03 21:58:44 UTC (rev 92315)
@@ -24,7 +24,6 @@
 
 module window {
     interface [
-        LegacyDefaultOptionalArguments,
         Conditional=MEDIA_STREAM,
         Callback=FunctionOnly
     ] NavigatorUserMediaSuccessCallback {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to