947286 - [Exif Parser] Consolidate all global variables in a single one

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Gallery, enhancement)

Description

[Diego Marcos [:dmarcos]]

The Exif parser now exposes several APIs in several global variables. We have to consolidate all the APIs under a single global.

Comment 1

[Diego Marcos [:dmarcos]]

Attachment: Pull Request

Comment 2

[David Flanagan [:djf]]

Comment on attachment 8343896 [details] [review]
Pull Request

Looks okay to me.  I left a few nits on github.  (added them to the jpeg.js file instead of the individual component files that make it up.)

Before landing, please:

- squash your commits
- make sure gjslint passes
- consider renaming JPEG.BlobView to JPEG.RWBlobView to indicate that it is different than BlobView
- consider changing the names jpeg.js and JPEG to jpeg_metadata.js and JPEGMetadata so it is clear that this is not a library that manipulates the actual jpeg image data.

Comment 3

[Diego Marcos [:dmarcos]]

Merged in master:

https://github.com/mozilla-b2g/exif-parser/commit/f82a0de0da1fb0d9a6eb808dcd84e0b43e76d06c