summaryrefslogtreecommitdiffstats
path: root/chrome/browser/download/download_history.cc
diff options
context:
space:
mode:
authorestade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-10-07 21:59:14 +0000
committerestade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-10-07 21:59:14 +0000
commit43ce682440911e31d0365e66e8911494434581d5 (patch)
treec2ae9d2b98d91dff4e4526ec981204930f2552f8 /chrome/browser/download/download_history.cc
parent912fee76d9a90bf184f1b985a4dcc1898c3fbe6c (diff)
downloadchromium_src-43ce682440911e31d0365e66e8911494434581d5.zip
chromium_src-43ce682440911e31d0365e66e8911494434581d5.tar.gz
chromium_src-43ce682440911e31d0365e66e8911494434581d5.tar.bz2
Remove all FIXMEs in browser/
Changed them to TODO or NOTE or other as the situation called for it. BUG=NONE TEST=NONE (no code change) Review URL: http://codereview.chromium.org/3599021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61861 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/download/download_history.cc')
-rw-r--r--chrome/browser/download/download_history.cc10
1 files changed, 5 insertions, 5 deletions
diff --git a/chrome/browser/download/download_history.cc b/chrome/browser/download/download_history.cc
index 9c86708..dd945c5 100644
--- a/chrome/browser/download/download_history.cc
+++ b/chrome/browser/download/download_history.cc
@@ -52,7 +52,7 @@ void DownloadHistory::AddEntry(
// handles, so we use a negative value. Eventually, they could overlap, but
// you'd have to do enough downloading that your ISP would likely stab you in
// the neck first. YMMV.
- // FIXME(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
+ // TODO(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
HistoryService* hs = profile_->GetHistoryService(Profile::EXPLICIT_ACCESS);
if (download_item->is_otr() || download_item->is_extension_install() ||
download_item->is_temporary() || !hs) {
@@ -71,7 +71,7 @@ void DownloadHistory::UpdateEntry(DownloadItem* download_item) {
if (download_item->db_handle() <= kUninitializedHandle)
return;
- // FIXME(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
+ // TODO(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
HistoryService* hs = profile_->GetHistoryService(Profile::EXPLICIT_ACCESS);
if (!hs)
return;
@@ -87,7 +87,7 @@ void DownloadHistory::UpdateDownloadPath(DownloadItem* download_item,
if (download_item->db_handle() <= kUninitializedHandle)
return;
- // FIXME(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
+ // TODO(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
HistoryService* hs = profile_->GetHistoryService(Profile::EXPLICIT_ACCESS);
if (hs)
hs->UpdateDownloadPath(new_path, download_item->db_handle());
@@ -98,7 +98,7 @@ void DownloadHistory::RemoveEntry(DownloadItem* download_item) {
if (download_item->db_handle() <= kUninitializedHandle)
return;
- // FIXME(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
+ // TODO(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
HistoryService* hs = profile_->GetHistoryService(Profile::EXPLICIT_ACCESS);
if (hs)
hs->RemoveDownload(download_item->db_handle());
@@ -106,7 +106,7 @@ void DownloadHistory::RemoveEntry(DownloadItem* download_item) {
void DownloadHistory::RemoveEntriesBetween(const base::Time remove_begin,
const base::Time remove_end) {
- // FIXME(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
+ // TODO(paulg) see bug 958058. EXPLICIT_ACCESS below is wrong.
HistoryService* hs = profile_->GetHistoryService(Profile::EXPLICIT_ACCESS);
if (hs)
hs->RemoveDownloadsBetween(remove_begin, remove_end);