Refactors to remove stream.js dependency on colorspace.js #7134
Conversation
yurydelendik
force-pushed
the
circ-stream-colorspace
branch
from
fcb6965
to
4740df8
Compare
|
The title/commit message says "Refactors to remove image.js dependency on colorspace.js", but shouldn't this rather be "Refactors to remove stream.js dependency on colorspace.js", given the code changes (there's still a Image/ColorSpace dependency left)? Note that @brendandahl suggested a slight name change, see http://logs.glob.uno/?c=mozilla%23pdfjs&s=31+Mar+2016&e=31+Mar+2016#c48005. I don't have a strong opinion either way, so change this if you see fit. r=me, with the above addressed, and passing tests. |
yurydelendik
changed the title
yurydelendik
force-pushed
the
circ-stream-colorspace
branch
from
4740df8
to
35cbf74
Compare
|
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.22.172.223:8877/30be6a76fd935f1/output.txt |
From: Bot.io (Linux)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.21.233.14:8877/4bbcee57efe086a/output.txt |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/30be6a76fd935f1/output.txt Total script time: 20.19 mins
|
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/4bbcee57efe086a/output.txt Total script time: 21.78 mins
|
Partially addresses #6777
Shall we use isNativelySupported instead of isNativelyDecodable ?