Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
9,371 Commits 1 Branch 80 Tags 254 MiB
fc2d8c15e8
Commit Graph

9371 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Yury Delendik
fc2d8c15e8 Adds noopener and nofollow to rel attribute of hyperlinks. 2017-01-23 10:34:27 -06:00
Yury Delendik
a9cf281723 Fixes IE9 document.readyState and input.type=number issues. 2017-01-23 09:23:13 -06:00
Tim van der Meij
1948a53ebb Merge pull request #7973 from Snuffleupagus/eslint_spaced-comment 
Enable the `spaced-comment` ESLint rule
 2017-01-22 21:58:42 +01:00
Tim van der Meij
17dd2e6b20 Merge pull request #7979 from Snuffleupagus/eslint-more-rules 
Enable the `no-unsafe-finally`/`no-octal`/`no-useless-call` ESLint rules
 2017-01-22 20:53:26 +01:00
Jonas Jenwald
54d55e8b7a Merge pull request #7980 from mukulmishra18/lint-errors 
Fix #7978: Fixes ESLint yoda rule for the URL polyfill.
 2017-01-22 00:49:20 +01:00
Mukul Mishra
4e38200030 Fix #7978: Fixes ESLint yoda rule for the URL polyfill. 2017-01-21 22:47:28 +05:30
Jonas Jenwald
82ea7e6e6e Enable the no-unsafe-finally/no-octal/no-useless-call ESLint rules 
http://eslint.org/docs/rules/no-unsafe-finally, there's just one violation which in this case can actually be ignored since there's nothing `return`ed there.
http://eslint.org/docs/rules/no-octal, there're no violations in the code-base.
http://eslint.org/docs/rules/no-useless-call, there's just one violation that needs to be fixed.
 2017-01-21 17:15:57 +01:00
Tim van der Meij
f8e793f8c2 Merge pull request #7974 from Snuffleupagus/eslint_no-lone-blocks 
Enable the `no-lone-blocks` ESLint rule
 2017-01-19 22:35:38 +01:00
Jonas Jenwald
31684e6918 Enable the no-lone-blocks ESLint rule 
http://eslint.org/docs/rules/no-lone-blocks

Note that we currently have no code that violates this rule in the source files, but it seems that the built files are possibly affected (see issue 7957).
 2017-01-19 19:56:23 +01:00
Jonas Jenwald
4626fc8342 Enable the spaced-comment ESLint rule 
Please see http://eslint.org/docs/rules/spaced-comment.

Note that the exceptions added for `line` comments are intended to still allow use of the old preprocessor without linting errors.
Also, I took the opportunity to improve the grammar slightly (w.r.t. capitalization and punctuation) for comments touched in the patch.
 2017-01-19 16:41:59 +01:00
Tim van der Meij
a917443ae6 Merge pull request #7965 from Snuffleupagus/bug-1331081 
[Bug 1331081] omit addEventListener/removeEventListener's third parameter when it's false
 2017-01-17 22:26:26 +01:00
Jonas Jenwald
70c1a6b120 [Bug 1331081] omit addEventListener/removeEventListener's third parameter when it's false 
Upstream changes from https://bugzilla.mozilla.org/show_bug.cgi?id=1331081; this patch also covers one file, `pdf_print_service.js`, that's not present in mozilla-central.

Fixes 7962.
 2017-01-17 17:50:21 +01:00
Tim van der Meij
c1703e977d Merge pull request #7960 from Snuffleupagus/eslint_space-unary-ops 
Adjust the `space-unary-ops` ESLint rule to comply with mozilla-central lint rules
 2017-01-16 22:18:39 +01:00
Jonas Jenwald
0dff8f3600 Adjust the space-unary-ops ESLint rule to comply with mozilla-central lint rules 
See http://eslint.org/docs/rules/space-unary-ops; a *very* small part of issue 7957.
 2017-01-16 17:19:25 +01:00
Jonas Jenwald
c0a47fddcc Update l10n files 2017-01-16 09:54:50 +01:00
Yury Delendik
9b111c9157 Merge pull request #7956 from Snuffleupagus/bug-1330147 
[Bug 1330147] removeObserver calls should not supply a third parameter
 2017-01-13 08:06:18 -06:00
Jonas Jenwald
54a64863e2 [Bug 1330147] removeObserver calls should not supply a third parameter 
*This is yet another very recent change that we were not notified about.*

Upstream changes from https://bugzilla.mozilla.org/show_bug.cgi?id=1330147, specifically https://hg.mozilla.org/mozilla-central/rev/8e0b7bc5fd71.
 2017-01-13 11:34:26 +01:00
Yury Delendik
76b4c8fae1 Merge pull request #7954 from yurydelendik/bug1329182 
[Bug 1329182] remove trailing newURI null parameters
 2017-01-12 19:41:44 -06:00
Yury Delendik
1f424e561d [Bug 1329182] remove trailing newURI null parameters 2017-01-12 19:40:51 -06:00
Yury Delendik
b8cd14336e Merge pull request #7913 from Snuffleupagus/addon-minimum-firefox45 
[Firefox addon] Change the minimum supported version to Firefox 45, i.e. the current ESR version, and remove no longer necessary fallback code
 2017-01-12 15:12:27 -06:00
Tim van der Meij
1fda987a4c Merge pull request #7904 from Snuffleupagus/issue-7901 
Further adjust the heuristics used to detect OpenType font files with CFF data, to ensure that all Type0 fonts are handled the same way regardless of font Subtype (issue 7901)
 2017-01-12 21:55:57 +01:00
Yury Delendik
c0d7029039 Merge pull request #7951 from Snuffleupagus/FileAttachmentAnnotation-simplified-unittest 
Rename `annotation_layer_spec.js` to `annotation_spec.js` to better describe what is actually tested, and simplify the `FileAttachmentAnnotation` unit-test to avoid having to use the entire API in the test
 2017-01-12 12:27:26 -06:00
Jonas Jenwald
e88c9c75db Simplify the FileAttachmentAnnotation unit-test to avoid having to use the entire API in the test 
Every other unit-test in `annotation_spec.js` is already only testing the annotation code. Hence it seems unnecessarily convoluted to make use of the API here, when we can (fairly) simply provide the necessary data explicitly as in all the other annotation unit-test.
 2017-01-12 19:10:37 +01:00
Jonas Jenwald
7c3a63bae0 Rename annotation_layer_spec.js to annotation_spec.js since the unit-tests only cover src/core/annotation.js functionality 2017-01-12 19:08:57 +01:00
Yury Delendik
1af35a6d88 Merge pull request #7948 from Skaty/fix-bidi-fraction 
Fix inverted calculation of RTL text percentage in bidi
 2017-01-12 10:40:08 -06:00
Syed Abdullah
857a5da8f1 Fix inverted calculation of RTL text percentage in bidi. 2017-01-12 23:54:06 +08:00
Jonas Jenwald
00ab0abd4e Merge pull request #7945 from yurydelendik/mv-locale-cmaps 
Moves locale and cmaps tasks to gulpfile.
 2017-01-11 14:05:21 +01:00
Yury Delendik
393740e2ae Merge pull request #7869 from PedroPachecoInf/master 
Fixes issue #6071 - TIFF with 1 bit-depth
 2017-01-10 12:37:26 -06:00
Yury Delendik
f1abd834d8 Moves locale and cmaps tasks to gulpfile. 2017-01-10 11:50:38 -06:00
jazzchipc
493853031b Fixes issue #6071. 
Corrects readBlockTiff() case for 1-bit depth and 1 color TIFF images incorporated in the PDF.

Adds reference test for PDF used to fix this issue.
 2017-01-10 16:42:43 +00:00
Jonas Jenwald
e5cea05881 Merge pull request #7944 from yurydelendik/rm-defpref 
Removes Promise usage from preferences.js
 2017-01-10 17:19:26 +01:00
Yury Delendik
cf7a9f99c8 Merge pull request #7943 from Snuffleupagus/fixtures-test-fixes 
Various fixes for the `external/builder/test*.js` files
 2017-01-10 08:58:37 -06:00
Yury Delendik
930a28d879 Better DOMContentLoaded handling. 2017-01-10 08:49:08 -06:00
Yury Delendik
ac2d4abf39 Removes web/default_preferences.js file. 2017-01-10 08:33:32 -06:00
Jonas Jenwald
3aa37ae8bc Add the external/builder/fixtures/ directory to .eslintignore, to avoid having to disable various lint rules locally 
This is similar to the already existing exception for `external/builder/fixtures_esprima`.
 2017-01-10 14:45:40 +01:00
Jonas Jenwald
fe9a561d45 Actually increment the errors counter on failures in test-fixtures.js/test_fixtures_esprima.js, so that the test runners won't incorrectly report that the tests passed 2017-01-10 14:45:36 +01:00
Jonas Jenwald
cf00516f04 Rename the external/builder/test*.js files to make it more obvious what they test 
Currently you have to open the files to be certain which tests each one will run, which we can avoid by appending the directory names to the file names of the tests themselves.
 2017-01-10 14:27:05 +01:00
Jonas Jenwald
ca0ebdfa56 Merge pull request #7942 from yurydelendik/rm-deadcode 
Fixes preprocessor testing and adds deadcode removal.
 2017-01-10 13:31:47 +01:00
Yury Delendik
77b7b84d1e Removes rest of 'no-else-return' comments. 2017-01-09 19:13:36 -06:00
Yury Delendik
6265bb6038 Fixes preprocessor testing and adds deadcode removal. 2017-01-09 19:05:36 -06:00
Tim van der Meij
f828f07ccd Merge pull request #7941 from Snuffleupagus/Page-idFactory 
Replace direct lookup of `uniquePrefix`/`idCounters`, in `Page` instances, with an `idFactory` containing an `createObjId` method instead
 2017-01-10 00:25:48 +01:00
Jonas Jenwald
642d8621ef Replace direct lookup of uniquePrefix/idCounters, in Page instances, with an idFactory containing an createObjId method instead 
We're currently making use of `uniquePrefix`/`idCounters` in multiple files, to create unique object id's, and adding a new occurrence of them requires some care to ensure that an object id isn't accidentally reused.
Furthermore, having to pass around multiple parameters as we currently do seem like something you want to avoid.

Instead, this patch adds a factory which means that there's only *one* thing that needs to be passed around. And since it's now only necessary to call a method in order to obtain a unique object id, the details are thus abstracted away at the call-sites which avoids accidental reuse of object id's.

To test that this works as expected a very simple `Page` unit-test is added, and the existing `Annotation layer` tests are also adjusted slightly.
 2017-01-09 23:16:25 +01:00
Tim van der Meij
e259bc2c16 Merge pull request #7939 from yurydelendik/unittesttravis 
Enables some unit tests on travis.
 2017-01-09 23:14:17 +01:00
Tim van der Meij
098acb1269 Merge pull request #7935 from Snuffleupagus/eslint_no-else-return 
Enable the `no-else-return` ESLint rule
 2017-01-09 23:03:48 +01:00
Yury Delendik
c45300e06c Enables some unit tests on travis. 2017-01-09 15:43:45 -06:00
Jonas Jenwald
4046d67fde Enable the no-else-return ESLint rule 
Using `else` after `return` is not necessary, and can often lead to unnecessarily cluttered code. By using the `no-else-return` rule in ESLint we can avoid this pattern, see http://eslint.org/docs/rules/no-else-return.
 2017-01-09 20:27:39 +01:00
Yury Delendik
049d7fa277 Merge pull request #7934 from porlan1/master 
Unit test files as modules
 2017-01-09 11:27:31 -06:00
porlan1
d9e1cb7955 unit test files as UMD modules 2017-01-09 11:40:57 -05:00
Jonas Jenwald
15f75a5585 [Firefox addon] Remove the registerAddonHistogram fallback code for Firefox versions 36 (and below) from PdfJsTelemetry-addon.jsm 
Given that the addon doesn't even work in Firefox versions prior to 38, this is just dead code that can now be removed.
 2017-01-07 12:19:42 +01:00
Jonas Jenwald
9434c023ab [Firefox addon] Change the minimum supported version to Firefox 45, i.e. the current ESR version, and remove no longer necessary fallback code 
According to https://wiki.mozilla.org/RapidRelease/Calendar#Past_branch_dates: The *last* ESR version of Firefox 38 was released in April this year, and since June the only available ESR version has been based on Firefox 45.

Now that Seamonkey has *finally* released a new version, i.e. 2.46 which should correspond to Firefox 49, there doesn't seem to be any reason to keep the fallback code around in the addon anymore.
 2017-01-07 12:19:42 +01:00