Don't use a magic value for 'auto' oriented <marker>s

Don't use a magic value for 'auto' oriented <marker>s

The value -1 (degrees) is a valid angle, so using it to indicate that
'auto' orientation should be used does not work out.
Just check 'orientType' directly instead and simplify the angle getter.

BUG=606345
Committed: https://crrev.com/886546446df947816927a7757de45fab1646c919
Cr-Commit-Position: refs/heads/master@{#390029}

[fs, 2016-04-26 17:21:00]

fs@opera.com changed reviewers:
+ fmalita@chromium.org, schenney@chromium.org

[Stephen Chennney, 2016-04-26 17:45:01]

lgtm. I wish we could magically find all the horrid code that still exists.

[fs, 2016-04-26 18:05:05]

On 2016/04/26 at 17:45:01, schenney wrote:
> lgtm. I wish we could magically find all the horrid code that still exists.

=)

[fs, 2016-04-26 18:05:11]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-04-26 18:05:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1916173003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1916173003/1

[commit-bot: I haz the power, 2016-04-26 20:02:27]

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

[commit-bot: I haz the power, 2016-04-26 20:02:28]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[fs, 2016-04-27 09:06:45]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-04-27 09:06:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1916173003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1916173003/1

[commit-bot: I haz the power, 2016-04-27 09:13:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-27 09:15:18]

Description was changed from

==========
Don't use a magic value for 'auto' oriented <marker>s

The value -1 (degrees) is a valid angle, so using it to indicate that
'auto' orientation should be used does not work out.
Just check 'orientType' directly instead and simplify the angle getter.

BUG=606345
==========

to

==========
Don't use a magic value for 'auto' oriented <marker>s

The value -1 (degrees) is a valid angle, so using it to indicate that
'auto' orientation should be used does not work out.
Just check 'orientType' directly instead and simplify the angle getter.

BUG=606345

Committed: https://crrev.com/886546446df947816927a7757de45fab1646c919
Cr-Commit-Position: refs/heads/master@{#390029}
==========

[commit-bot: I haz the power, 2016-04-27 09:15:19]

Patchset 1 (id:??) landed as
https://crrev.com/886546446df947816927a7757de45fab1646c919
Cr-Commit-Position: refs/heads/master@{#390029}