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

Unified Diff: chrome/browser/android/tab_base_android_impl.cc

Issue 10968003: Add rendering support to the TestShell. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed Nits Created 8 years, 3 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: chrome/browser/android/tab_base_android_impl.cc
diff --git a/chrome/browser/android/tab_base_android_impl.cc b/chrome/browser/android/tab_base_android_impl.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b3834c32b18e99065d52024a62b1d3fba92a3404
--- /dev/null
+++ b/chrome/browser/android/tab_base_android_impl.cc
@@ -0,0 +1,138 @@
+// 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 "chrome/browser/android/tab_base_android_impl.h"
+
+#include "base/android/jni_string.h"
+#include "base/logging.h"
+#include "chrome/browser/android/chrome_web_contents_delegate_android.h"
+#include "chrome/browser/net/url_fixer_upper.h"
+#include "content/public/browser/android/content_view_core.h"
+#include "content/public/browser/web_contents.h"
+#include "googleurl/src/gurl.h"
+#include "jni/TabBase_jni.h"
+#include "third_party/WebKit/Source/Platform/chromium/public/WebLayer.h"
+
+using base::android::ConvertJavaStringToUTF8;
+using base::android::ConvertUTF8ToJavaString;
+using base::android::ScopedJavaLocalRef;
+using chrome::android::ChromeWebContentsDelegateAndroid;
+using content::WebContents;
+
+namespace {
+class ChromeWebContentsDelegateRenderAndroid
+ : public ChromeWebContentsDelegateAndroid {
+ public:
+ ChromeWebContentsDelegateRenderAndroid(TabBaseAndroidImpl* tab_android_impl,
+ JNIEnv* env, jobject obj)
sky 2012/09/25 15:57:39 each param on own line.
David Trainor- moved to gerrit 2012/09/25 19:07:12 Done.
+ : ChromeWebContentsDelegateAndroid(env, obj),
+ tab_android_impl_(tab_android_impl) {
+ }
+
+ virtual ~ChromeWebContentsDelegateRenderAndroid() {
+ }
+
+ virtual void AttachLayer(WebContents* web_contents,
+ WebKit::WebLayer* layer) OVERRIDE {
+ tab_android_impl_->tab_layer()->addChild(layer);
+ }
+
+ virtual void RemoveLayer(WebContents* web_contents,
+ WebKit::WebLayer* layer) OVERRIDE {
+ layer->removeFromParent();
+ }
+
+ private:
+ TabBaseAndroidImpl* tab_android_impl_;
+};
sky 2012/09/25 15:57:39 DISALLOW_...
David Trainor- moved to gerrit 2012/09/25 19:07:12 Done.
+} // namespace
+
+TabBaseAndroidImpl::TabBaseAndroidImpl(JNIEnv* env,
+ jobject obj,
+ WebContents* web_contents)
+ : web_contents_(web_contents),
+ tab_layer_(WebKit::WebLayer::create()) {
+}
+
+TabBaseAndroidImpl::~TabBaseAndroidImpl() {
+}
+
+void TabBaseAndroidImpl::Destroy(JNIEnv* env, jobject obj) {
+ delete this;
+}
+
+browser_sync::SyncedTabDelegate* TabBaseAndroidImpl::GetSyncedTabDelegate() {
+ NOTIMPLEMENTED();
+ return NULL;
+}
+
+void TabBaseAndroidImpl::OnReceivedHttpAuthRequest(jobject auth_handler,
+ const string16& host,
+ const string16& realm) {
+ NOTIMPLEMENTED();
+}
+
+void TabBaseAndroidImpl::ShowContextMenu(
+ const content::ContextMenuParams& params) {
+ NOTIMPLEMENTED();
+}
+
+void TabBaseAndroidImpl::ShowCustomContextMenu(
+ const content::ContextMenuParams& params,
+ const base::Callback<void(int)>& callback) {
+ NOTIMPLEMENTED();
+}
+
+void TabBaseAndroidImpl::ShowSelectFileDialog(
+ const base::android::ScopedJavaLocalRef<jobject>& select_file) {
+ NOTIMPLEMENTED();
+}
+
+void TabBaseAndroidImpl::AddShortcutToBookmark(
+ const GURL& url, const string16& title, const SkBitmap& skbitmap,
+ int r_value, int g_value, int b_value) {
+ NOTIMPLEMENTED();
+}
+
+void TabBaseAndroidImpl::RunExternalProtocolDialog(const GURL& url) {
+ NOTIMPLEMENTED();
+}
+
+bool TabBaseAndroidImpl::RegisterTabBaseAndroidImpl(JNIEnv* env) {
+ return RegisterNativesImpl(env);
+}
+
+void TabBaseAndroidImpl::InitWebContentsDelegate(
+ JNIEnv* env,
+ jobject obj,
+ jobject web_contents_delegate) {
+ web_contents_delegate_.reset(
+ new ChromeWebContentsDelegateRenderAndroid(this,
+ env,
+ web_contents_delegate));
+ web_contents_->SetDelegate(web_contents_delegate_.get());
+}
+
+ScopedJavaLocalRef<jstring> TabBaseAndroidImpl::FixupUrl(JNIEnv* env,
+ jobject obj,
+ jstring url) {
+ GURL fixed_url(URLFixerUpper::FixupURL(ConvertJavaStringToUTF8(env, url),
+ std::string()));
+
+ std::string fixed_spec;
+ if (fixed_url.is_valid())
+ fixed_spec = fixed_url.spec();
+
+ return ConvertUTF8ToJavaString(env, fixed_spec);
+}
+
+static jint Init(JNIEnv* env,
+ jobject obj,
+ jint web_contents_ptr) {
+ TabBaseAndroidImpl* tab = new TabBaseAndroidImpl(
+ env,
+ obj,
+ reinterpret_cast<WebContents*>(web_contents_ptr));
+ return reinterpret_cast<jint>(tab);
+}

Powered by Google App Engine
This is Rietveld 408576698