Fix for DCHECK on SmbService initialization

This removes the initialization of TempFileManager in SmbService.
With this change, Mount will be in a broken state and will be fixed in a
future CL.

Bug: 837492
Change-Id: I07689fd5a19e2bf56aac5d2f6cd9085d2352dcad
Reviewed-on: https://chromium-review.googlesource.com/1040678
Commit-Queue: Allen Vicencio <allenvic@chromium.org>
Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org>
Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555570}

chrome/browser/chromeos/smb_client/smb_service.cc

@@ -40,11 +40,14 @@ SmbService* SmbService::Get(content::BrowserContext* context) {
 void SmbService::Mount(const file_system_provider::MountOptions& options,
                        const base::FilePath& share_path,
                        MountResponse callback) {
+  // TODO(allenvic): Temporary workaround for https://crbug.com/837492.
+  CHECK(false);
+
   // TODO(allenvic): Implement passing of credentials. This currently passes
   // empty credentials to SmbProvider.
   GetSmbProviderClient()->Mount(
       share_path, "" /* workgroup */, "" /* username */,
-      temp_file_manager_.WritePasswordToFile("" /* password */),
+      temp_file_manager_->WritePasswordToFile("" /* password */),
       base::BindOnce(&SmbService::OnMountResponse, AsWeakPtr(),
                      base::Passed(&callback), options, share_path));
 }

chrome/browser/chromeos/smb_client/smb_service.h

@@ -85,7 +85,7 @@ class SmbService : public KeyedService,
 
   const ProviderId provider_id_;
   Profile* profile_;
-  TempFileManager temp_file_manager_;
+  std::unique_ptr<TempFileManager> temp_file_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(SmbService);
 };