Skip to content
This repository has been archived by the owner on Jun 29, 2018. It is now read-only.

Fix Bug 1038392 - Image plugin test urls are no longer reachable #404

Merged
merged 2 commits into from
Dec 17, 2014

Conversation

georgemarshall
Copy link
Contributor

@@ -123,12 +123,12 @@ asyncTest( "size test", 4, function() {
popped.image({
start: 0,
end: 3,
src: "https://www.drumbeat.org/media/images/drumbeat-logo-splash.png",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make all these protocol less, example:

src: "//www.drumbeat.org/media/images/drumbeat-logo-splash.png",

Would that fix this too?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The actual path for the images has changed, as well as https no longer being available.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wow, I wasn't looking close enough!

Yeah, this makes sense to me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fun to see the old drumbeat logo kicking around :)

ScottDowne added a commit that referenced this pull request Dec 17, 2014
Fix Bug 1038392 - Image plugin test urls are no longer reachable
@ScottDowne ScottDowne merged commit 19cb6b0 into mozilla:master Dec 17, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants