Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(241)

Side by Side Diff: chrome/browser/importer/external_process_importer_host.cc

Issue 9549014: Coverity: No uninitialized fields in constructor. (CID 9446) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/importer/external_process_importer_host.h" 5 #include "chrome/browser/importer/external_process_importer_host.h"
6 6
7 #include "chrome/browser/bookmarks/bookmark_model.h" 7 #include "chrome/browser/bookmarks/bookmark_model.h"
8 #include "chrome/browser/importer/external_process_importer_client.h" 8 #include "chrome/browser/importer/external_process_importer_client.h"
9 #include "chrome/browser/importer/in_process_importer_bridge.h" 9 #include "chrome/browser/importer/in_process_importer_bridge.h"
10 10
11 ExternalProcessImporterHost::ExternalProcessImporterHost() 11 ExternalProcessImporterHost::ExternalProcessImporterHost()
12 : items_(0), 12 : client_(NULL),
13 source_profile_(NULL),
14 items_(0),
13 cancelled_(false), 15 cancelled_(false),
14 import_process_launched_(false) { 16 import_process_launched_(false) {
15 } 17 }
16 18
17 void ExternalProcessImporterHost::Cancel() { 19 void ExternalProcessImporterHost::Cancel() {
18 cancelled_ = true; 20 cancelled_ = true;
19 if (import_process_launched_) 21 if (import_process_launched_)
20 client_->Cancel(); 22 client_->Cancel();
21 NotifyImportEnded(); // Tells the observer that we're done, and releases us. 23 NotifyImportEnded(); // Tells the observer that we're done, and releases us.
22 } 24 }
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 66
65 void ExternalProcessImporterHost::Loaded(BookmarkModel* model, 67 void ExternalProcessImporterHost::Loaded(BookmarkModel* model,
66 bool ids_reassigned) { 68 bool ids_reassigned) {
67 DCHECK(model->IsLoaded()); 69 DCHECK(model->IsLoaded());
68 model->RemoveObserver(this); 70 model->RemoveObserver(this);
69 waiting_for_bookmarkbar_model_ = false; 71 waiting_for_bookmarkbar_model_ = false;
70 installed_bookmark_observer_ = false; 72 installed_bookmark_observer_ = false;
71 73
72 InvokeTaskIfDone(); 74 InvokeTaskIfDone();
73 } 75 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698