OLD | NEW |
---|---|
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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/in_process_importer_bridge.h" | 5 #include "chrome/browser/importer/in_process_importer_bridge.h" |
6 | 6 |
7 #include "base/bind.h" | 7 #include "base/bind.h" |
8 #include "base/file_util.h" | 8 #include "base/file_util.h" |
9 #include "base/strings/utf_string_conversions.h" | 9 #include "base/strings/utf_string_conversions.h" |
10 #include "chrome/browser/importer/external_process_importer_host.h" | 10 #include "chrome/browser/importer/external_process_importer_host.h" |
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
47 importer::VisitSource visit_source) { | 47 importer::VisitSource visit_source) { |
48 switch (visit_source) { | 48 switch (visit_source) { |
49 case importer::VISIT_SOURCE_BROWSED: | 49 case importer::VISIT_SOURCE_BROWSED: |
50 return history::SOURCE_BROWSED; | 50 return history::SOURCE_BROWSED; |
51 case importer::VISIT_SOURCE_FIREFOX_IMPORTED: | 51 case importer::VISIT_SOURCE_FIREFOX_IMPORTED: |
52 return history::SOURCE_FIREFOX_IMPORTED; | 52 return history::SOURCE_FIREFOX_IMPORTED; |
53 case importer::VISIT_SOURCE_IE_IMPORTED: | 53 case importer::VISIT_SOURCE_IE_IMPORTED: |
54 return history::SOURCE_IE_IMPORTED; | 54 return history::SOURCE_IE_IMPORTED; |
55 case importer::VISIT_SOURCE_SAFARI_IMPORTED: | 55 case importer::VISIT_SOURCE_SAFARI_IMPORTED: |
56 return history::SOURCE_SAFARI_IMPORTED; | 56 return history::SOURCE_SAFARI_IMPORTED; |
57 } | 57 }; |
58 NOTREACHED(); | 58 NOTREACHED(); |
gab
2013/07/09 20:48:31
I think these last 2 lines can be removed.
scottmg
2013/07/09 21:16:45
I think they should be kept in case new enums are
gab
2013/07/09 21:29:43
Well if they are removed and new enums are added,
scottmg
2013/07/09 23:03:09
I don't believe that's true. e.g. win vs2010 relea
| |
59 return history::SOURCE_SYNCED; | 59 return history::SOURCE_SYNCED; |
60 } | 60 } |
61 | 61 |
62 } // namespace | 62 } // namespace |
63 | 63 |
64 using content::BrowserThread; | 64 using content::BrowserThread; |
65 | 65 |
66 namespace { | 66 namespace { |
67 | 67 |
68 // FirefoxURLParameterFilter is used to remove parameter mentioning Firefox from | 68 // FirefoxURLParameterFilter is used to remove parameter mentioning Firefox from |
(...skipping 207 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
276 BrowserThread::PostTask( | 276 BrowserThread::PostTask( |
277 BrowserThread::UI, FROM_HERE, | 277 BrowserThread::UI, FROM_HERE, |
278 base::Bind(&ExternalProcessImporterHost::NotifyImportEnded, host_)); | 278 base::Bind(&ExternalProcessImporterHost::NotifyImportEnded, host_)); |
279 } | 279 } |
280 | 280 |
281 string16 InProcessImporterBridge::GetLocalizedString(int message_id) { | 281 string16 InProcessImporterBridge::GetLocalizedString(int message_id) { |
282 return l10n_util::GetStringUTF16(message_id); | 282 return l10n_util::GetStringUTF16(message_id); |
283 } | 283 } |
284 | 284 |
285 InProcessImporterBridge::~InProcessImporterBridge() {} | 285 InProcessImporterBridge::~InProcessImporterBridge() {} |
OLD | NEW |