Commit Graph

9 Commits

Author SHA1 Message Date
Rob Wu
f67a66c108 Disable Chrome extension for non-GET requests 2013-07-27 11:26:55 +02:00
Rob Wu
ec5ef58b84 [CRX] Improved navigation detection.
A user reported that the PDF Viewer is not rendered on Dropbox,
(Chrome on Mac OS X). This is apparently caused by the fact that the
PDF file is loaded in an iframe in such a way that the tabs.onUpdated
event is not triggered.

This patch switches to the webNavigation event API, which improves the
reliability of the navigation detection.

Unfortunately Opera 15 does not support the webNavigation API, so the
old (tabs.onUpdated) method is used (feature-detection is used, so
whenever Opera decides to implement this API, it will profit from it).
2013-07-17 19:04:10 +02:00
Rob Wu
be012ed1df [CRX] Don't intercept download of PDF attachment
Fixes #3426
2013-07-01 19:17:05 +02:00
Rob Wu
efcf2aed6e [Chrome extension] Added JSDocs to pdfHandler.js
And a small refactor to reduce the LOC in the onHeadersReceived
callback, to improve the readability of the code.
2013-07-01 19:16:45 +02:00
Rob Wu
e181a3c902 Highly improved Chrome extension
Full list feature changes in this commit:
- Support for iframes
- Switched to content-type (MIME) detection instead of hard-coding a
  case-sensitive check for the .PDF extension
- The PDF's original URL is visible in the omnibox
- Support for incognito mode

Note: PDF viewer is disabled for the file:// + incognito
combination, because it's currently impossible to get the combination
to work.

See https://github.com/mozilla/pdf.js/pull/3017#issuecomment-15693432
2013-04-04 10:04:13 +02:00
Yury Delendik
fcc169453b Adds the extensions/* files for jshint 2013-02-02 21:15:43 -06:00
Michael Payne
05bca47b11 Include uppercase extensions for Chrome
add uppercase case extension recognition for Chrome pdf.js
2012-10-17 21:37:24 -07:00
Michael Payne
6ca9245a39 Changes to allowed versioned building of Chrome extension that meets new
Chrome extension security requirements
2012-09-06 12:39:49 -07:00
Michael Payne
088c6e17e8 changes to allow chrome extension to load
changes to remove inline scripts, update manifest version, fix make.js
so compatibility.js isn't included for chrome. Due to new Chrome
extension changes outlined at
http://developer.chrome.com/extensions/manifestVersion.html
2012-09-05 22:52:17 -07:00