Remove DownloadItem::Is*() in favor of DI::GetState()

content/browser/android/download_controller_android_impl.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 "content/browser/android/download_controller_android_impl.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 236 matching lines @@
       ConvertUTF8ToJavaString(env, download_item->GetMimeType());
   ScopedJavaLocalRef<jstring> jfilename = ConvertUTF8ToJavaString(
       env, download_item->GetTargetFilePath().BaseName().value());
   Java_DownloadController_onDownloadStarted(
       env, GetJavaObject()->Controller(env).obj(), view.obj(), jfilename.obj(),
       jmime_type.obj());
 }
 
 void DownloadControllerAndroidImpl::OnDownloadUpdated(DownloadItem* item) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  if (item->IsDangerous() && !item->IsCancelled())
+  if (item->IsDangerous() &&
+      (item->GetState() != DownloadItem::CANCELLED))
     OnDangerousDownload(item);
 
-  if (!item->IsComplete())
+  if (item->GetState() != DownloadItem::COMPLETE)
     return;
 
   // Multiple OnDownloadUpdated() notifications may be issued while the download
   // is in the COMPLETE state. Only handle one.
   item->RemoveObserver(this);
 
   // Call onDownloadCompleted
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jstring> jurl =
       ConvertUTF8ToJavaString(env, item->GetURL().spec());
@@ skipping 110 matching lines @@
     referer = referer_url.spec();
   if (!request->url_chain().empty()) {
     original_url = request->url_chain().front();
     url = request->url_chain().back();
   }
 }
 
 DownloadControllerAndroidImpl::DownloadInfoAndroid::~DownloadInfoAndroid() {}
 
 }  // namespace content