Refactoring the code for location manager.

Code refactoring for location provider while registering for location updates.
While registering the location provider for location updates location updates are requested
redundantly for gps enabled or disabled code.

BUG=411182

Review URL: https://codereview.chromium.org/546633004

Cr-Commit-Position: refs/heads/master@{#293766}

content/public/android/java/src/org/chromium/content/browser/LocationProviderFactory.java

@@ -150,13 +150,9 @@ public class LocationProviderFactory {
             // bounce notifications to the Geolocation thread as they arrive in the mainLooper.
             try {
                 Criteria criteria = new Criteria();
+                if (isGpsEnabled) criteria.setAccuracy(Criteria.ACCURACY_FINE);
                 mLocationManager.requestLocationUpdates(0, 0, criteria, this,
                         ThreadUtils.getUiThreadLooper());
-                if (isGpsEnabled) {
-                    criteria.setAccuracy(Criteria.ACCURACY_FINE);
-                    mLocationManager.requestLocationUpdates(0, 0, criteria, this,
-                            ThreadUtils.getUiThreadLooper());
-                }
             } catch (SecurityException e) {
                 Log.e(TAG, "Caught security exception registering for location updates from " +
                     "system. This should only happen in DumpRenderTree.");