Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-09-15 17:38:23 +01:00
Code Issues Projects Releases Packages Wiki Activity

Auto merge of #15285 - simartin:issue_15017, r=nox

Browse source 
Issue #15017: Properly handle and report network errors on page loads.

<!-- Please describe your changes on the following line: -->
This patch fixes how network errors are handled during page loads: they would not be reported and cause crashes before, and do not anymore.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #15017 (github issue number if applicable).

<!-- Either: -->
- [X] There are tests for these changes

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/15285)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2017-02-03 06:04:40 -08:00 • committed by GitHub
commit f0e21d0bbe
3 changed files with 36 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

6
tests/wpt/mozilla/meta/MANIFEST.json
View file

@ -8786,6 +8786,12 @@
"url": "/_mozilla/mozilla/nested_asap_script.html"
}
],
"mozilla/network_error_page_load.html": [
{
"path": "mozilla/network_error_page_load.html",
"url": "/_mozilla/mozilla/network_error_page_load.html"
}
],
"mozilla/node_compareDocumentPosition.html": [
{
"path": "mozilla/node_compareDocumentPosition.html",

13
tests/wpt/mozilla/tests/mozilla/network_error_page_load.html Normal file
View file

@ -0,0 +1,13 @@
<!doctype html>
<meta charset="utf-8">
<title>Test for issue #15017</title>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<html>
<iframe src="http://aowiejfoiawjef" id="foo"></iframe>
</html>
<script>
var t = async_test("Load resource with network error")
var iframe = document.getElementById('foo')
iframe.onload = t.step_func(function(e) { t.done() })
</script>