Delete the PNaCl translate thread before deleting object files on surfaway.

ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc

 // Copyright (c) 2012 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 "ppapi/native_client/src/trusted/plugin/pnacl_coordinator.h"
 
 #include <utility>
 #include <vector>
 
 #include "native_client/src/include/checked_cast.h"
@@ skipping 235 matching lines @@
   // will have been destroyed.  This will result in the cancellation of
   // translation_complete_callback_, so no notification will be delivered.
   if (translate_thread_.get() != NULL) {
     translate_thread_->AbortSubprocesses();
   }
   if (!translation_finished_reported_) {
     plugin_->nacl_interface()->ReportTranslationFinished(
         plugin_->pp_instance(),
         PP_FALSE);
   }
+  // Force deleting the translate_thread now. It must be deleted
+  // before any scoped_* fields hanging off of PnaclCoordinator
+  // since the thread may be accessing those fields.
+  // It will also be accessing obj_files_.
+  translate_thread_.reset(NULL);
+  // TODO(jvoung): use base/memory/scoped_vector.h to hold obj_files_.
   for (int i = 0; i < num_object_files_opened_; i++) {
     delete obj_files_[i];
   }
 }
 
 nacl::DescWrapper* PnaclCoordinator::ReleaseTranslatedFD() {
   DCHECK(temp_nexe_file_ != NULL);
   return temp_nexe_file_->release_read_wrapper();
 }
 
@@ skipping 416 matching lines @@
                                   temp_nexe_file_.get(),
                                   invalid_desc_wrapper_.get(),
                                   &error_info_,
                                   resources_.get(),
                                   &pnacl_options_,
                                   this,
                                   plugin_);
 }
 
 }  // namespace plugin