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

Unified Diff: blimp/client/android/blimp_toolbar.h

Issue 1422363008: Add a basic UI to the Android Blimp client. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unused icon Created 5 years, 1 month 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: blimp/client/android/blimp_toolbar.h
diff --git a/blimp/client/android/blimp_toolbar.h b/blimp/client/android/blimp_toolbar.h
new file mode 100644
index 0000000000000000000000000000000000000000..a9f88bcb6886f5f57c31c010958300080620f093
--- /dev/null
+++ b/blimp/client/android/blimp_toolbar.h
@@ -0,0 +1,49 @@
+// Copyright 2015 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.
+
+#ifndef BLIMP_CLIENT_ANDROID_BLIMP_TOOLBAR_H_
+#define BLIMP_CLIENT_ANDROID_BLIMP_TOOLBAR_H_
+
+#include "base/android/jni_android.h"
+#include "base/macros.h"
+
+class GURL;
+class SkBitmap;
+
+namespace blimp {
+
+// The native component of org.chromium.blimp.toolbar.BlimpToolbar. This
+// handles marshalling calls between Java and native. Specifically, this passes
+// calls between BlimpToolbar.java <=> content_lite's NavigationController
+// layer.
+class BlimpToolbar {
+ public:
+ static bool RegisterJni(JNIEnv* env);
+
+ BlimpToolbar(JNIEnv* env, const base::android::JavaParamRef<jobject>& jobj);
+
+ // Methods called from Java via JNI.
+ void Destroy(JNIEnv* env, jobject jobj);
+ void OnUrlTextEntered(JNIEnv* env, jobject jobj, jstring text);
+ void OnReloadPressed(JNIEnv* env, jobject jobj);
+ jboolean OnBackPressed(JNIEnv* env, jobject jobj);
+
+ // To be called by a the content lite NavigationController. Should probably
nyquist 2015/11/09 01:58:45 Nit: Should this be a TODO?
David Trainor- moved to gerrit 2015/11/09 17:17:45 Done.
+ // be an overridden delegate method so the network code can be multi platform.
+ void OnNavigationStateChanged(const GURL* url,
+ const SkBitmap* favicon,
+ const std::string* title);
+
+ private:
+ virtual ~BlimpToolbar();
+
+ // Reference to the Java object which owns this class.
+ base::android::ScopedJavaGlobalRef<jobject> java_obj_;
+
+ DISALLOW_COPY_AND_ASSIGN(BlimpToolbar);
+};
+
+} // namespace blimp
+
+#endif // BLIMP_CLIENT_ANDROID_BLIMP_TOOLBAR_H_

Powered by Google App Engine
This is Rietveld 408576698