Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trinity College Dublin Image Conformance Project (N.B not fully complete yet) #9021

Closed
wants to merge 11 commits into from

Conversation

@paterson
Copy link

paterson commented Dec 18, 2015

Hi there!

We're the group behind this feature:

https://github.com/servo/servo/wiki/Image-load-conformance-student-project

We've completed the initial tasks as outlined above, except for one issue with conformance with JSTrace, HeapSizeOf etc. We've outlined the scenario here:

https://groups.google.com/forum/#!topic/mozilla.dev.servo/HvgCJ9-ozns

We may complete the project after Christmas on our own accord, but we just needed to have a pull request submitted as part of our college module, so this is why we are submitting this now. It is not ready for consideration quite yet. We will update this pull request with any further progress.

Thanks!

Review on Reviewable

TomasBarry and others added 10 commits Nov 11, 2015
Added the readonly currentSrc attribute to HTMLImageElement.webidl and
implemented the CurrentSrc getter in HTMLImageElement.rs
Added the crossOrigin attribute to HTMLImageElement.webidl and a
make_setter and make_getter for crossOrigin to HTMLImageElement.rs
Trinity College Task 1 (Update htmlimagerequest.rs)
Trinity College Dublin Image Conformance 2nd Task (Add crossOrigin attribute to HTMLImageElement.webidl)
Merged Trinity College Dublin image conformance project 3rd task (Add currentSrc attribute to HTMLImageElement.webidl)
Merged Trinity College Dublin's Image Conformance project (4th task). add a command for the image cache task that accepts a URL and a vector of bytes. This command should instruct the cache to store this data as a newly-complete network request and continue decoding the result into pixel data.
@highfive
Copy link

highfive commented Dec 18, 2015

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @metajack (or someone else) soon.

@paterson paterson changed the title Trinity College Dublin Image Conformance Project (N.B not fully complete) Trinity College Dublin Image Conformance Project (N.B not fully complete yet) Dec 18, 2015
@jdm
Copy link
Member

jdm commented Dec 18, 2015

Hi @paterson! Did my reply on Dec 10 help with the scenario you mention?

@KiChjang
Copy link
Member

KiChjang commented Jan 6, 2016

@paterson Ping?

@jdm jdm self-assigned this Jan 6, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Jan 10, 2016

The latest upstream changes (presumably #9214) made this pull request unmergeable. Please resolve the merge conflicts.

@KiChjang
Copy link
Member

KiChjang commented Feb 1, 2016

Rebased as #9480.

@KiChjang KiChjang closed this Feb 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
You can’t perform that action at this time.