uncaught exception in dom/base/test/test_viewsource_forbidden_in_object.html

RESOLVED FIXED in Firefox 53

Status

()

--
trivial
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mats, Assigned: bobowen)

Tracking

Trunk
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
I just noticed this while looking through a mochitest log:

> INFO - TEST-START | dom/base/test/test_viewsource_forbidden_in_object.html
> INFO - JavaScript error: http://mochi.test:8888/tests/dom/base/test/test_viewsource_forbidden_in_object.html, line 19: uncaught exception: 2152398858
> INFO - JavaScript error: http://mochi.test:8888/tests/dom/base/test/test_viewsource_forbidden_in_object.html, line 19: uncaught exception: 2152398858
> INFO - JavaScript error: resource://app/components/FeedConverter.js, line 516: uncaught exception: 2152398858
> INFO - JavaScript error: resource://app/components/FeedConverter.js, line 516: NS_ERROR_MALFORMED_URI: The URI is malformed'The URI is malformed' when calling method: [nsIProtocolHandler::newURI]

Is the uncaught exception expected?
or is there something wrong with the test?

https://treeherder.mozilla.org/logviewer.html#?job_id=67088661&repo=try&lineNumber=4885
(In reply to Mats Palmgren (:mats) from comment #0)
> I just noticed this while looking through a mochitest log:
> 
> > INFO - TEST-START | dom/base/test/test_viewsource_forbidden_in_object.html
> > INFO - JavaScript error: http://mochi.test:8888/tests/dom/base/test/test_viewsource_forbidden_in_object.html, line 19: uncaught exception: 2152398858
> > INFO - JavaScript error: http://mochi.test:8888/tests/dom/base/test/test_viewsource_forbidden_in_object.html, line 19: uncaught exception: 2152398858
> > INFO - JavaScript error: resource://app/components/FeedConverter.js, line 516: uncaught exception: 2152398858
> > INFO - JavaScript error: resource://app/components/FeedConverter.js, line 516: NS_ERROR_MALFORMED_URI: The URI is malformed'The URI is malformed' when calling method: [nsIProtocolHandler::newURI]
> 
> Is the uncaught exception expected?
> or is there something wrong with the test?
> 

Bob is the author of the test case. Let's see if he has answers.

> https://treeherder.mozilla.org/logviewer.
> html#?job_id=67088661&repo=try&lineNumber=4885
Flags: needinfo?(bobowencode)
(Assignee)

Comment 2

2 years ago
Created attachment 8826166 [details] [diff] [review]
Remove object view-source tests that are no longer valid URIs

I guess these errors started when Bug 1277685 landed.
I think just removing those cases makes sense, since they're no longer valid URIs so no longer test this code.

MozReview-Commit-ID: FSuz0zhVUNv
Attachment #8826166 - Flags: review?(bzbarsky)
(Assignee)

Updated

2 years ago
Assignee: nobody → bobowencode
Status: NEW → ASSIGNED
(Assignee)

Updated

2 years ago
Flags: needinfo?(bobowencode)
Comment on attachment 8826166 [details] [diff] [review]
Remove object view-source tests that are no longer valid URIs

>-      desc: "Test 4: pcast:view-source should not be allowed in an object",
>-      URI: "pcast:view-source:file_general_document.html"

Why is his one going away?  Doe sit use the feed converter too?

r=me if so, r=me with it restored if it should have stayed.
Attachment #8826166 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 4

2 years ago
Thanks.

(In reply to Boris Zbarsky [:bz] (still a bit busy) from comment #3)
 
> >-      desc: "Test 4: pcast:view-source should not be allowed in an object",
> >-      URI: "pcast:view-source:file_general_document.html"
> 
> Why is his one going away?  Doe sit use the feed converter too?

Yes it goes through FeedConverter.js as well.
(Assignee)

Comment 5

2 years ago
This only removes some now invalid test cases, so I don't think it is worth pushing to try.
I have run the test locally.
Keywords: checkin-needed

Comment 6

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f3ccf0f50160
Remove object view-source tests that are no longer valid URIs. r=bz
Keywords: checkin-needed

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f3ccf0f50160
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.