Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[WIP] Add concept of screens to engine #19425

Closed
wants to merge 8 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 9 additions & 2 deletions ci/licenses_golden/licenses_flutter
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,9 @@ FILE: ../../../flutter/lib/ui/ui_benchmarks.cc
FILE: ../../../flutter/lib/ui/ui_dart_state.cc
FILE: ../../../flutter/lib/ui/ui_dart_state.h
FILE: ../../../flutter/lib/ui/window.dart
FILE: ../../../flutter/lib/ui/window/platform_configuration.cc
FILE: ../../../flutter/lib/ui/window/platform_configuration.h
FILE: ../../../flutter/lib/ui/window/platform_configuration_unittests.cc
FILE: ../../../flutter/lib/ui/window/platform_message.cc
FILE: ../../../flutter/lib/ui/window/platform_message.h
FILE: ../../../flutter/lib/ui/window/platform_message_response.cc
Expand All @@ -408,6 +411,10 @@ FILE: ../../../flutter/lib/ui/window/pointer_data_packet.h
FILE: ../../../flutter/lib/ui/window/pointer_data_packet_converter.cc
FILE: ../../../flutter/lib/ui/window/pointer_data_packet_converter.h
FILE: ../../../flutter/lib/ui/window/pointer_data_packet_converter_unittests.cc
FILE: ../../../flutter/lib/ui/window/screen.cc
FILE: ../../../flutter/lib/ui/window/screen.h
FILE: ../../../flutter/lib/ui/window/screen_metrics.cc
FILE: ../../../flutter/lib/ui/window/screen_metrics.h
FILE: ../../../flutter/lib/ui/window/viewport_metrics.cc
FILE: ../../../flutter/lib/ui/window/viewport_metrics.h
FILE: ../../../flutter/lib/ui/window/window.cc
Expand Down Expand Up @@ -571,6 +578,8 @@ FILE: ../../../flutter/runtime/dart_vm_unittests.cc
FILE: ../../../flutter/runtime/embedder_resources.cc
FILE: ../../../flutter/runtime/embedder_resources.h
FILE: ../../../flutter/runtime/fixtures/runtime_test.dart
FILE: ../../../flutter/runtime/platform_data.cc
FILE: ../../../flutter/runtime/platform_data.h
FILE: ../../../flutter/runtime/ptrace_ios.cc
FILE: ../../../flutter/runtime/ptrace_ios.h
FILE: ../../../flutter/runtime/runtime_controller.cc
Expand All @@ -583,8 +592,6 @@ FILE: ../../../flutter/runtime/skia_concurrent_executor.cc
FILE: ../../../flutter/runtime/skia_concurrent_executor.h
FILE: ../../../flutter/runtime/test_font_data.cc
FILE: ../../../flutter/runtime/test_font_data.h
FILE: ../../../flutter/runtime/window_data.cc
FILE: ../../../flutter/runtime/window_data.h
FILE: ../../../flutter/shell/common/animator.cc
FILE: ../../../flutter/shell/common/animator.h
FILE: ../../../flutter/shell/common/animator_unittests.cc
Expand Down
34 changes: 32 additions & 2 deletions examples/glfw/FlutterEmbedderGLFW.cc
Original file line number Diff line number Diff line change
Expand Up @@ -68,14 +68,43 @@ static void GLFWKeyCallback(GLFWwindow* window,
}

void GLFWwindowSizeCallback(GLFWwindow* window, int width, int height) {
int left, top;
glfwGetWindowPos(window, &left, &top);
FlutterWindowMetricsEvent event = {};
event.struct_size = sizeof(event);
event.window_id = 0;
event.left = left * g_pixelRatio;
event.top = top * g_pixelRatio;
event.width = width * g_pixelRatio;
event.height = height * g_pixelRatio;
event.pixel_ratio = g_pixelRatio;

// TODO(gspencergoog): Currently, there is only one window. This will change
// as multi-window support is added. See
// https://github.com/flutter/flutter/issues/60131
FlutterEngineSendWindowMetricsEvent(
reinterpret_cast<FlutterEngine>(glfwGetWindowUserPointer(window)), &event,
1);
}

void GLFWwindowPosCallback(GLFWwindow* window, int left, int top) {
int width, height;
glfwGetWindowSize(window, &width, &height);
FlutterWindowMetricsEvent event = {};
event.struct_size = sizeof(event);
event.window_id = 0;
event.left = left * g_pixelRatio;
event.top = top * g_pixelRatio;
event.width = width * g_pixelRatio;
event.height = height * g_pixelRatio;
event.pixel_ratio = g_pixelRatio;

// TODO(gspencergoog): Currently, there is only one window. This will change
// as multi-window support is added. See
// https://github.com/flutter/flutter/issues/60131
FlutterEngineSendWindowMetricsEvent(
reinterpret_cast<FlutterEngine>(glfwGetWindowUserPointer(window)),
&event);
reinterpret_cast<FlutterEngine>(glfwGetWindowUserPointer(window)), &event,
1);
}

bool RunFlutter(GLFWwindow* window,
Expand Down Expand Up @@ -150,6 +179,7 @@ int main(int argc, const char* argv[]) {

glfwSetKeyCallback(window, GLFWKeyCallback);
glfwSetWindowSizeCallback(window, GLFWwindowSizeCallback);
glfwSetWindowPosCallback(window, GLFWwindowPosCallback);
glfwSetMouseButtonCallback(window, GLFWmouseButtonCallback);

while (!glfwWindowShouldClose(window)) {
Expand Down
7 changes: 7 additions & 0 deletions lib/ui/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,8 @@ source_set_maybe_fuchsia_legacy("ui") {
"text/text_box.h",
"ui_dart_state.cc",
"ui_dart_state.h",
"window/platform_configuration.cc",
"window/platform_configuration.h",
"window/platform_message.cc",
"window/platform_message.h",
"window/platform_message_response.cc",
Expand All @@ -101,6 +103,10 @@ source_set_maybe_fuchsia_legacy("ui") {
"window/pointer_data_packet.h",
"window/pointer_data_packet_converter.cc",
"window/pointer_data_packet_converter.h",
"window/screen.cc",
"window/screen.h",
"window/screen_metrics.cc",
"window/screen_metrics.h",
"window/viewport_metrics.cc",
"window/viewport_metrics.h",
"window/window.cc",
Expand Down Expand Up @@ -180,6 +186,7 @@ if (enable_unittests) {
sources = [
"painting/image_encoding_unittests.cc",
"painting/vertices_unittests.cc",
"window/platform_configuration_unittests.cc",
"window/pointer_data_packet_converter_unittests.cc",
]

Expand Down
9 changes: 8 additions & 1 deletion lib/ui/compositing/scene.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
#include "flutter/lib/ui/painting/image.h"
#include "flutter/lib/ui/painting/picture.h"
#include "flutter/lib/ui/ui_dart_state.h"
#include "flutter/lib/ui/window/platform_configuration.h"
#include "flutter/lib/ui/window/window.h"
#include "third_party/skia/include/core/SkImageInfo.h"
#include "third_party/skia/include/core/SkSurface.h"
Expand Down Expand Up @@ -41,7 +42,13 @@ Scene::Scene(std::shared_ptr<flutter::Layer> rootLayer,
uint32_t rasterizerTracingThreshold,
bool checkerboardRasterCacheImages,
bool checkerboardOffscreenLayers) {
auto viewport_metrics = UIDartState::Current()->window()->viewport_metrics();
// TODO(gspencergoog): Currently, there is only one window. This will change
// as multi-window support is added. See
// https://github.com/flutter/flutter/issues/60131
auto viewport_metrics = UIDartState::Current()
->platform_configuration()
->window(0)
->viewport_metrics();

layer_tree_ = std::make_unique<LayerTree>(
SkISize::Make(viewport_metrics.physical_width,
Expand Down
4 changes: 2 additions & 2 deletions lib/ui/dart_ui.cc
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
#include "flutter/lib/ui/text/font_collection.h"
#include "flutter/lib/ui/text/paragraph.h"
#include "flutter/lib/ui/text/paragraph_builder.h"
#include "flutter/lib/ui/window/window.h"
#include "flutter/lib/ui/window/platform_configuration.h"
#include "third_party/tonic/converter/dart_converter.h"
#include "third_party/tonic/logging/dart_error.h"

Expand Down Expand Up @@ -81,7 +81,7 @@ void DartUI::InitForGlobal() {
SemanticsUpdate::RegisterNatives(g_natives);
SemanticsUpdateBuilder::RegisterNatives(g_natives);
Vertices::RegisterNatives(g_natives);
Window::RegisterNatives(g_natives);
PlatformConfiguration::RegisterNatives(g_natives);
#if defined(LEGACY_FUCHSIA_EMBEDDER)
SceneHost::RegisterNatives(g_natives);
#endif
Expand Down
Loading