From 9075224c779d95ad2c604fd73cab4ad2fe5e545e Mon Sep 17 00:00:00 2001 From: Emmanuel Garcia Date: Wed, 24 Jun 2020 11:01:30 -0700 Subject: [PATCH] Move OnDisplayPlatformView JNI call (#19258) --- .../external_view_embedder.cc | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/shell/platform/android/external_view_embedder/external_view_embedder.cc b/shell/platform/android/external_view_embedder/external_view_embedder.cc index f1f46ab21b4..048e96295f6 100644 --- a/shell/platform/android/external_view_embedder/external_view_embedder.cc +++ b/shell/platform/android/external_view_embedder/external_view_embedder.cc @@ -91,16 +91,6 @@ bool AndroidExternalViewEmbedder::SubmitFrame( for (size_t i = 0; i < composition_order_.size(); i++) { int64_t view_id = composition_order_[i]; - SkRect view_rect = GetViewRect(view_id); - - // Display the platform view. If it's already displayed, then it's - // just positioned and sized. - jni_facade_->FlutterViewOnDisplayPlatformView(view_id, // - view_rect.x(), // - view_rect.y(), // - view_rect.width(), // - view_rect.height() // - ); sk_sp picture = picture_recorders_.at(view_id)->finishRecordingAsPicture(); @@ -156,6 +146,15 @@ bool AndroidExternalViewEmbedder::SubmitFrame( frame->Submit(); for (int64_t view_id : composition_order_) { + SkRect view_rect = GetViewRect(view_id); + // Display the platform view. If it's already displayed, then it's + // just positioned and sized. + jni_facade_->FlutterViewOnDisplayPlatformView(view_id, // + view_rect.x(), // + view_rect.y(), // + view_rect.width(), // + view_rect.height() // + ); for (const SkRect& overlay_rect : overlay_layers.at(view_id)) { CreateSurfaceIfNeeded(context, // view_id, //