Fix the indirect pure virtual calls in DataFetcherSharedMemoryBase destructor.

content/browser/device_sensors/data_fetcher_shared_memory_base.cc

 // Copyright 2014 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 "content/browser/device_sensors/data_fetcher_shared_memory_base.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/threading/thread.h"
@@ skipping 86 matching lines @@
   fetcher_->Fetch(consumers_bitmask_);
 }
 
 // --- end of PollingThread methods
 
 DataFetcherSharedMemoryBase::DataFetcherSharedMemoryBase()
     : started_consumers_(0) {
 }
 
 DataFetcherSharedMemoryBase::~DataFetcherSharedMemoryBase() {
-  StopFetchingDeviceData(CONSUMER_TYPE_MOTION);
-  StopFetchingDeviceData(CONSUMER_TYPE_ORIENTATION);
+  DCHECK(started_consumers_ == 0);

[Bernhard Bauer, 2014/09/17 16:41:00]

If you use DCHECK_EQ() (with the expected value first!), you'll get a nicer
error message on failure.

[timvolodine, 2014/09/17 18:25:00]

right, done.

 
   // make sure polling thread stops asap.
   if (polling_thread_)
     polling_thread_->Stop();
 
   STLDeleteContainerPairSecondPointers(shared_memory_map_.begin(),
       shared_memory_map_.end());
 }
 
 bool DataFetcherSharedMemoryBase::StartFetchingDeviceData(
@@ skipping 37 matching lines @@
   } else {
     if (!Stop(consumer_type))
       return false;
   }
 
   started_consumers_ ^= consumer_type;
 
   return true;
 }
 
+void DataFetcherSharedMemoryBase::StopFetchingAllDeviceData() {
+  StopFetchingDeviceData(CONSUMER_TYPE_MOTION);
+  StopFetchingDeviceData(CONSUMER_TYPE_ORIENTATION);
+}
+
 base::SharedMemoryHandle
 DataFetcherSharedMemoryBase::GetSharedMemoryHandleForProcess(
     ConsumerType consumer_type, base::ProcessHandle process) {
   SharedMemoryMap::const_iterator it = shared_memory_map_.find(consumer_type);
   if (it == shared_memory_map_.end())
     return base::SharedMemory::NULLHandle();
 
   base::SharedMemoryHandle renderer_handle;
   it->second->ShareToProcess(process, &renderer_handle);
   return renderer_handle;
@@ skipping 57 matching lines @@
 }
 
 base::MessageLoop* DataFetcherSharedMemoryBase::GetPollingMessageLoop() const {
   return polling_thread_ ? polling_thread_->message_loop() : NULL;
 }
 
 bool DataFetcherSharedMemoryBase::IsPollingTimerRunningForTesting() const {
   return polling_thread_ ? polling_thread_->IsTimerRunning() : false;
 }
 
-
 }  // namespace content