Makes use of EngineGeolocationFeature weak_ptr threadsafe.

blimp/engine/feature/geolocation/engine_geolocation_feature_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "blimp/engine/feature/geolocation/engine_geolocation_feature.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 
@@ skipping 92 matching lines @@
         location_provider_(feature_.CreateGeolocationDelegate()
                                ->OverrideSystemLocationProvider()),
         mock_callback_(
             base::Bind(&EngineGeolocationFeatureTest::OnLocationUpdate,
                        base::Unretained(this))) {}
 
   void SetUp() override {
     out_processor_ = new MockBlimpMessageProcessor();
     feature_.set_outgoing_message_processor(base::WrapUnique(out_processor_));
     location_provider_->SetUpdateCallback(mock_callback_);
+    base::RunLoop().RunUntilIdle();
   }
 
   void OnLocationUpdate(const device::LocationProvider* provider,
                         const device::Geoposition& geoposition) {
     received_position = geoposition;
   }
 
  protected:
   // This is a raw pointer to a class that is owned by the GeolocationFeature.
   MockBlimpMessageProcessor* out_processor_;
@@ skipping 48 matching lines @@
   EXPECT_CALL(*out_processor_,
               MockableProcessMessage(
                   EqualsUpdatedRequestLevel(
                       GeolocationSetInterestLevelMessage::NO_INTEREST),
                   _))
       .Times(1);
 
   location_provider_->StartProvider(true);
   location_provider_->StartProvider(false);
   location_provider_->StopProvider();
+  base::RunLoop().RunUntilIdle();
 }
 
 TEST_F(EngineGeolocationFeatureTest, UnexpectedMessageReceived) {
   std::unique_ptr<BlimpMessage> message = CreateUnexpectedMessage();
   net::TestCompletionCallback cb;
   feature_.ProcessMessage(std::move(message), cb.callback());
 
   EXPECT_EQ(net::ERR_UNEXPECTED, cb.WaitForResult());
 }
 
 TEST_F(EngineGeolocationFeatureTest,
        OnPermissionGrantedTriggersRefreshRequest) {
   EXPECT_CALL(*out_processor_,
               MockableProcessMessage(
                   EqualsUpdatedRequestLevel(
                       GeolocationSetInterestLevelMessage::HIGH_ACCURACY),
                   _))
       .Times(1);
 
   EXPECT_CALL(*out_processor_,
               MockableProcessMessage(EqualsRequestRefresh(), _))
       .Times(1);
-  EXPECT_CALL(*out_processor_,
-              MockableProcessMessage(
-                  EqualsUpdatedRequestLevel(
-                      GeolocationSetInterestLevelMessage::NO_INTEREST),
-                  _))
-      .Times(1);
 
   location_provider_->StartProvider(true);
   location_provider_->OnPermissionGranted();
+  base::RunLoop().RunUntilIdle();
 }
 
 }  // namespace engine
 }  // namespace blimp