fix tests

Author: Chris Yang

shell/common/rasterizer.cc

@@ -473,8 +473,7 @@ RasterStatus Rasterizer::DrawToSurface(flutter::LayerTree& layer_tree) {
       FML_DCHECK(!frame->IsSubmitted());
       external_view_embedder->SubmitFrame(surface_->GetContext(),
                                           std::move(frame));
-    }
-    if (!frame->IsSubmitted()) {
+    } else {
       frame->Submit();
     }
 

shell/platform/darwin/ios/framework/Source/FlutterPlatformViews.mm

@@ -473,7 +473,7 @@
   // Any UIKit related code has to run on main thread.
   FML_DCHECK([[NSThread currentThread] isMainThread]);
   if (flutter_view_ == nullptr) {
-    return false;
+    return frame->Submit();
   }
 
   DisposeViews();

shell/platform/darwin/ios/framework/Source/FlutterPlatformViewsTest.mm

@@ -5,6 +5,7 @@
 #import <OCMock/OCMock.h>
 #import <XCTest/XCTest.h>
 
+#include "flutter/flow/surface_frame.h"
 #import "flutter/shell/platform/darwin/common/framework/Headers/FlutterBinaryMessenger.h"
 #import "flutter/shell/platform/darwin/common/framework/Headers/FlutterMacros.h"
 #import "flutter/shell/platform/darwin/ios/framework/Headers/FlutterPlatformViews.h"
@@ -645,7 +646,10 @@ - (void)testFlutterPlatformViewControllerSubmitFrameWithoutFlutterViewNotCrashin
 
   flutterPlatformViewsController->PrerollCompositeEmbeddedView(2, std::move(embeddedViewParams));
   flutterPlatformViewsController->CompositeEmbeddedView(2);
-  flutterPlatformViewsController->SubmitFrame(nullptr, nullptr, nullptr);
+  auto mock_surface = std::make_unique<flutter::SurfaceFrame>(
+      nullptr, true,
+      [](const flutter::SurfaceFrame& surface_frame, SkCanvas* canvas) { return true; });
+  flutterPlatformViewsController->SubmitFrame(nullptr, nullptr, std::move(mock_surface));
 
   flutterPlatformViewsController->Reset();
 }