Expose heading and missing geolocation properties

[hansemannn]

Motivation

There is no core method to receive location heading updates. In addition, some of the geolocation event properties were missing as well, I added those, too.

Test Plan

/**
 * Sample React Native App
 * https://github.com/facebook/react-native
 * @flow
 */

import React, { Component } from 'react';
import {
  AppRegistry,
  StyleSheet,
  Text,
  View
} from 'react-native';

import { NativeModules } from 'react-native';

export default class test_storekit extends Component {
 
  componentDidMount() {
    navigator.geolocation.startUpdatingHeading(
      (heading) => {
        var currentHeading = JSON.stringify(heading);
        alert(currentHeading);
      }
    );
  }

  componentWillUnmount() {
    navigator.geolocation.stopUpdatingHeading();
  }
 
  render() {
    return (
      <View style={styles.container}>
        <Text style={styles.welcome}>
          Welcome to React Native!
        </Text>
        <Text style={styles.instructions}>
          To get started, edit index.ios.js
        </Text>
        <Text style={styles.instructions}>
          Press Cmd+R to reload,{'\n'}
          Cmd+D or shake for dev menu
        </Text>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  welcome: {
    fontSize: 20,
    textAlign: 'center',
    margin: 10,
  },
  instructions: {
    textAlign: 'center',
    color: '#333333',
    marginBottom: 5,
  },
});

AppRegistry.registerComponent('test_storekit', () => test_storekit);

[eslint-bot]

no-trailing-spaces: Trailing spaces not allowed.

[eslint-bot]

no-trailing-spaces: Trailing spaces not allowed.

[eslint-bot]

no-trailing-spaces: Trailing spaces not allowed.

[eslint-bot]

semi: Missing semicolon.

[eslint-bot]

no-trailing-spaces: Trailing spaces not allowed.

[eslint-bot]

no-trailing-spaces: Trailing spaces not allowed.

[eslint-bot]

no-trailing-spaces: Trailing spaces not allowed.

[shergin]

Should we deallocate _locationManager somewhere?

[shergin]

I would create a dedicated method for it like initLocationManagerIfNeeded.

[facebook-github-bot]

@hansemannn I tried to find reviewers for this pull request and wanted to ping them to take another look. However, based on the blame information for the files in this pull request I couldn't find any reviewers. This sometimes happens when the files in the pull request are new or don't exist on master anymore. Is this pull request still relevant? If yes could you please rebase? In case you know who has context on this code feel free to mention them in a comment (one person is fine). Thanks for reading and hope you will continue contributing to the project.

[shergin]

@hansemannn Btw, I still can review and merge your PR if you respond to my comments.