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

Unified Diff: blimp/client/app/android/blimp_compositor_manager_android.h

Issue 1925863003: Changed Blimp client to start with white screen before drawing contents (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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: blimp/client/app/android/blimp_compositor_manager_android.h
diff --git a/blimp/client/app/android/blimp_compositor_manager_android.h b/blimp/client/app/android/blimp_compositor_manager_android.h
index 2f9f6539532eea3257e48401cb217aeb2fe1934c..525f086b31a2f9dbda84ddb371abcfb939df1c02 100644
--- a/blimp/client/app/android/blimp_compositor_manager_android.h
+++ b/blimp/client/app/android/blimp_compositor_manager_android.h
@@ -7,6 +7,7 @@
#include "base/android/jni_android.h"
#include "base/macros.h"
+#include "blimp/client/app/android/blimp_view.h"
#include "blimp/client/feature/compositor/blimp_compositor_manager.h"
#include "ui/gfx/geometry/size.h"
#include "ui/gfx/native_widget_types.h"
@@ -39,6 +40,8 @@ class BlimpCompositorManagerAndroid : public BlimpCompositorManager {
~BlimpCompositorManagerAndroid() override;
+ void SetBlimpView(BlimpView* view) { blimp_view_ = view; }
+
protected:
// |size| is the size of the display. |real_size_supported| determines
// whether or not this size is the real display size or the display size
@@ -52,6 +55,8 @@ class BlimpCompositorManagerAndroid : public BlimpCompositorManager {
void GenerateLayerTreeSettings(cc::LayerTreeSettings* settings) override;
private:
+ void DidCompleteSwapBuffers() override;
+
// Used to determine tile size for the compositor's rastered tiles. For a
// device of width X height |portrait_width_| will be min(width, height) and
// |landscape_width_| will be max(width, height).
@@ -62,6 +67,9 @@ class BlimpCompositorManagerAndroid : public BlimpCompositorManager {
// physical dimensions, including any area occupied by system decorations.
bool real_size_supported_;
+ // Used to notify the BlimpView when compositor starts drawing to the screen.
+ BlimpView* blimp_view_;
Khushal 2016/04/28 03:32:26 I would suggest informing the BlimpView using a cl
shaktisahu 2016/04/28 21:38:15 Done.
+
DISALLOW_COPY_AND_ASSIGN(BlimpCompositorManagerAndroid);
};

Powered by Google App Engine
This is Rietveld 408576698