Feedback

Author: loic-sharma

shell/platform/windows/compositor_opengl.cc

@@ -78,6 +78,8 @@ bool CompositorOpenGL::CollectBackingStore(const FlutterBackingStore* store) {
 
 bool CompositorOpenGL::Present(const FlutterLayer** layers,
                                size_t layers_count) {
+  // TODO: Support compositing layers and platform views.
+  // See: https://github.com/flutter/flutter/issues/31713
   FML_DCHECK(is_initialized_);
   FML_DCHECK(layers_count == 1);
   FML_DCHECK(layers[0]->type == kFlutterLayerContentTypeBackingStore);

shell/platform/windows/compositor_opengl_unittests.cc

@@ -96,7 +96,7 @@ class CompositorOpenGLTest : public WindowsTest {
 
     engine_ = builder.Build();
     EngineModifier modifier(engine_.get());
-    modifier.SetSurfaceManager(surface_manager.release());
+    modifier.SetSurfaceManager(std::move(surface_manager));
   }
 
   void UseEngineWithView() {

shell/platform/windows/compositor_software.cc

@@ -39,6 +39,8 @@ bool CompositorSoftware::CollectBackingStore(const FlutterBackingStore* store) {
 
 bool CompositorSoftware::Present(const FlutterLayer** layers,
                                  size_t layers_count) {
+  // TODO: Support compositing layers and platform views.
+  // See: https://github.com/flutter/flutter/issues/31713
   FML_DCHECK(layers_count == 1);
   FML_DCHECK(layers[0]->type == kFlutterLayerContentTypeBackingStore);
   FML_DCHECK(layers[0]->backing_store->type ==