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

Unified Diff: content/browser/android/download_controller.cc

Issue 10693134: Revert 146000 - Split out ContentViewCore from ContentView for embedders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/android/download_controller.cc
===================================================================
--- content/browser/android/download_controller.cc (revision 146004)
+++ content/browser/android/download_controller.cc (working copy)
@@ -9,7 +9,6 @@
#include "base/bind.h"
#include "base/logging.h"
#include "base/memory/scoped_ptr.h"
-#include "content/browser/android/content_view_core_impl.h"
#include "content/browser/download/download_item_impl.h"
#include "content/browser/renderer_host/render_process_host_impl.h"
#include "content/browser/renderer_host/render_view_host_delegate.h"
@@ -219,7 +218,7 @@
// Register for updates to the DownloadItem.
download_item->AddObserver(this);
- jobject view = GetContentViewCoreFromWebContents(web_contents);
+ jobject view = GetContentViewFromWebContents(web_contents);
if(!view) {
// The view went away. Can't proceed.
return;
@@ -246,14 +245,14 @@
ScopedJavaLocalRef<jstring> jpath =
ConvertUTF8ToJavaString(env, item->GetFullPath().value());
- jobject view_core = GetContentViewCoreFromWebContents(item->GetWebContents());
- if (!view_core) {
+ jobject view = GetContentViewFromWebContents(item->GetWebContents());
+ if(!view) {
// We can get NULL WebContents from the DownloadItem.
return;
}
Java_DownloadController_onHttpPostDownloadCompleted(env,
- GetJavaObject()->Controller(env).obj(), view_core, jurl.obj(),
+ GetJavaObject()->Controller(env).obj(), view, jurl.obj(),
jcontent_disposition.obj(), jmime_type.obj(), jpath.obj(),
item->GetReceivedBytes(), true);
}
@@ -275,10 +274,10 @@
if (!web_contents)
return NULL;
- return GetContentViewCoreFromWebContents(web_contents);
+ return GetContentViewFromWebContents(web_contents);
}
-jobject DownloadController::GetContentViewCoreFromWebContents(
+jobject DownloadController::GetContentViewFromWebContents(
WebContents* web_contents) {
NOTIMPLEMENTED();
return NULL;
« no previous file with comments | « content/browser/android/download_controller.h ('k') | content/browser/renderer_host/render_widget_host_view_android.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698