Fix CR comments

Author: iskakaushik

shell/platform/embedder/embedder.cc

@@ -1012,24 +1012,22 @@ FlutterEngineResult FlutterEngineInitialize(size_t version,
                                 "embedder, got 0 active displays.");
     }
 
-    display_settings.displays =
-        new FlutterDisplay[display_settings.display_count];
+    std::vector<FlutterDisplay> displays(display_settings.display_count);
+    display_settings.displays = displays.data();
     display_settings_callback(user_data, &display_settings);
 
     // TODO(iskakaushik): Add support for multiple displays.
     main_display_refresh_rate = display_settings.displays[0].refresh_rate;
-
-    delete display_settings.displays;
   }
 
   flutter::PlatformViewEmbedder::PlatformDispatchTable platform_dispatch_table =
       {
-          update_semantics_nodes_callback,                //
-          update_semantics_custom_actions_callback,       //
-          platform_message_response_callback,             //
-          vsync_callback,                                 //
-          compute_platform_resolved_locale_callback,      //
-          static_cast<float>(main_display_refresh_rate),  //
+          update_semantics_nodes_callback,            //
+          update_semantics_custom_actions_callback,   //
+          platform_message_response_callback,         //
+          vsync_callback,                             //
+          compute_platform_resolved_locale_callback,  //
+          main_display_refresh_rate,                  //
       };
 
   auto on_create_platform_view = InferPlatformViewCreationCallback(

shell/platform/embedder/platform_view_embedder.h

@@ -39,7 +39,7 @@ class PlatformViewEmbedder final : public PlatformView {
     VsyncWaiterEmbedder::VsyncCallback vsync_callback;  // optional
     ComputePlatformResolvedLocaleCallback
         compute_platform_resolved_locale_callback;
-    float display_refresh_rate;
+    double display_refresh_rate;
   };
 
   // Creates a platform view that sets up an OpenGL rasterizer.