Ticket #2626 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

make Format.ArcXML tests pass in Safari

Reported by: bartvde Owned by:
Priority: minor Milestone: 2.10 Release
Component: Format.ArcXML Version: 2.9
Keywords: Cc:
State: Complete

Description

Parse errors are dealt with differently in Safari then the code now assumes.

Attachments

ticket2626.patch Download (471 bytes) - added by bartvde 3 years ago.

Change History

Changed 3 years ago by bartvde

Changed 3 years ago by bartvde

  • state set to Review

This one-line patch fixes up the test in Safari for ArcXML and ArcXML/Features, please review

Changed 3 years ago by elemoine

  • state changed from Review to Commit

Bart, could you please add a comment in the test to quickly explain why this extra check (arcNode.firstChild.nodeName === 'parsererror') is needed? Thanks.

Changed 3 years ago by bartvde

  • status changed from new to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [10302]) make Format.ArcXML tests pass in Safari, invalid XML documents are dealt with differently in Safari, r=elemoine (closes #2626)

Note: See TracTickets for help on using tickets.