Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Unified Diff: Source/core/xml/XSLTProcessor.idl

Issue 1222503003: Sync XML-related interfaces with their specs, or lack thereof (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/xml/XPathResult.idl ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/xml/XSLTProcessor.idl
diff --git a/Source/core/xml/XSLTProcessor.idl b/Source/core/xml/XSLTProcessor.idl
index 1198d41d5c5a515855b647a0221dfd15f8c32903..221628f55cf27fc645795d4eaab6a42e30697869 100644
--- a/Source/core/xml/XSLTProcessor.idl
+++ b/Source/core/xml/XSLTProcessor.idl
@@ -26,9 +26,9 @@
* THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-// Eventually we should implement XSLTException:
-// http://lxr.mozilla.org/seamonkey/source/content/xsl/public/nsIXSLTException.idl
-// http://bugs.webkit.org/show_bug.cgi?id=5446
+// There is no spec for XSLTProcessor. It was first implemented in Gecko and
+// their interface definition remains the closest thing to a spec:
+// https://hg.mozilla.org/mozilla-central/file/012853bd80b7/dom/webidl/XSLTProcessor.webidl
[
GarbageCollected,
@@ -39,11 +39,14 @@
TypeChecking=Interface,
] interface XSLTProcessor {
- void importStylesheet(Node stylesheet);
- // FIXME: These return null on failure; should throw exceptions instead.
- [CustomElementCallbacks] DocumentFragment? transformToFragment(Node source, Document docVal);
+ void importStylesheet(Node style);
+ // TODO(philipj): In Gecko, the transformTo*() methods throw an exception in
+ // case of error instead of returning null.
+ [CustomElementCallbacks] DocumentFragment? transformToFragment(Node source, Document output);
[CustomElementCallbacks] Document? transformToDocument(Node source);
+ // TODO(philipj): In Gecko, it's possible to set and get back any parameter
+ // value, not just DOMString.
void setParameter(DOMString? namespaceURI, DOMString localName, DOMString value);
DOMString? getParameter(DOMString? namespaceURI, DOMString localName);
void removeParameter(DOMString? namespaceURI, DOMString localName);
@@ -52,4 +55,3 @@
void reset();
};
-
« no previous file with comments | « Source/core/xml/XPathResult.idl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698