Add ImageOrientation class.

Add ImageOrientation class.

Currently we uses different number expressing image orientation for EXIF and
drive metadata.

The CL added generic ImageOrientation class that is going to replace the image
orientation numbers.

BUG=396702
TEST=ImageOrientationTest

Committed: https://crrev.com/cfc48a489802d33d25a3246c6b1f024df8c7e676
Cr-Commit-Position: refs/heads/master@{#321541}

[hirono, 2015-03-20 06:47:33]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-03-20 06:47:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1025593002/20001

[commit-bot: I haz the power, 2015-03-20 06:47:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-20 06:47:45]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[hirono, 2015-03-20 06:48:01]

hirono@chromium.org changed reviewers:
+ yawano@chromium.org

[hirono, 2015-03-20 06:48:02]

PTAL, thanks!

[yawano, 2015-03-20 07:35:42]

https://codereview.chromium.org/1025593002/diff/20001/ui/file_manager/file_ma...
File ui/file_manager/file_manager/foreground/js/metadata/image_orientation.js
(right):

https://codereview.chromium.org/1025593002/diff/20001/ui/file_manager/file_ma...
ui/file_manager/file_manager/foreground/js/metadata/image_orientation.js:62:
return new ImageOrientation(0, 1, 1, 0);
This matrix is same with case 5. (0, -1, -1, 0) for case 7?

[hirono, 2015-03-20 08:09:20]

Thanks!

https://codereview.chromium.org/1025593002/diff/20001/ui/file_manager/file_ma...
File ui/file_manager/file_manager/foreground/js/metadata/image_orientation.js
(right):

https://codereview.chromium.org/1025593002/diff/20001/ui/file_manager/file_ma...
ui/file_manager/file_manager/foreground/js/metadata/image_orientation.js:62:
return new ImageOrientation(0, 1, 1, 0);
On 2015/03/20 07:35:42, yawano wrote:
> This matrix is same with case 5. (0, -1, -1, 0) for case 7?

Yes, good catch! Done

[yawano, 2015-03-20 08:43:40]

lgtm. Thank you!

[hirono, 2015-03-20 09:38:50]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-03-20 09:39:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1025593002/40001

[commit-bot: I haz the power, 2015-03-20 10:12:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-20 10:14:00]

Patchset 3 (id:??) landed as
https://crrev.com/cfc48a489802d33d25a3246c6b1f024df8c7e676
Cr-Commit-Position: refs/heads/master@{#321541}