Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Merged
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
1 change: 1 addition & 0 deletions shell/platform/linux/fl_keyboard_handler.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
#include <gdk/gdk.h>

#include "flutter/shell/platform/linux/fl_keyboard_view_delegate.h"
#include "flutter/shell/platform/linux/public/flutter_linux/fl_binary_messenger.h"

G_BEGIN_DECLS

Expand Down
4 changes: 2 additions & 2 deletions shell/platform/linux/fl_keyboard_manager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -410,6 +410,7 @@ static void fl_keyboard_manager_init(FlKeyboardManager* self) {
}

FlKeyboardManager* fl_keyboard_manager_new(
FlBinaryMessenger* messenger,
FlKeyboardViewDelegate* view_delegate) {
g_return_val_if_fail(FL_IS_KEYBOARD_VIEW_DELEGATE(view_delegate), nullptr);

Expand All @@ -436,8 +437,7 @@ FlKeyboardManager* fl_keyboard_manager_new(
},
self)));
g_ptr_array_add(self->responder_list,
FL_KEY_RESPONDER(fl_key_channel_responder_new(
fl_keyboard_view_delegate_get_messenger(view_delegate))));
FL_KEY_RESPONDER(fl_key_channel_responder_new(messenger)));

return self;
}
Expand Down
3 changes: 3 additions & 0 deletions shell/platform/linux/fl_keyboard_manager.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
#include <gdk/gdk.h>

#include "flutter/shell/platform/linux/fl_keyboard_view_delegate.h"
#include "flutter/shell/platform/linux/public/flutter_linux/fl_binary_messenger.h"

G_BEGIN_DECLS

Expand Down Expand Up @@ -36,6 +37,7 @@ G_DECLARE_FINAL_TYPE(FlKeyboardManager,

/**
* fl_keyboard_manager_new:
* @messenger: an #FlBinaryMessenger.
* @view_delegate: An interface that the manager requires to communicate with
* the platform. Usually implemented by FlView.
*
Expand All @@ -44,6 +46,7 @@ G_DECLARE_FINAL_TYPE(FlKeyboardManager,
* Returns: a new #FlKeyboardManager.
*/
FlKeyboardManager* fl_keyboard_manager_new(
FlBinaryMessenger* messenger,
FlKeyboardViewDelegate* view_delegate);

/**
Expand Down
10 changes: 2 additions & 8 deletions shell/platform/linux/fl_keyboard_manager_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -330,12 +330,6 @@ static gboolean fl_mock_view_keyboard_text_filter_key_press(
return self->text_filter_result;
}

static FlBinaryMessenger* fl_mock_view_keyboard_get_messenger(
FlKeyboardViewDelegate* view_delegate) {
FlMockViewDelegate* self = FL_MOCK_VIEW_DELEGATE(view_delegate);
return FL_BINARY_MESSENGER(self->messenger);
}

static void fl_mock_view_keyboard_redispatch_event(
FlKeyboardViewDelegate* view_delegate,
FlKeyEvent* event) {
Expand All @@ -362,7 +356,6 @@ static void fl_mock_view_keyboard_delegate_iface_init(
FlKeyboardViewDelegateInterface* iface) {
iface->send_key_event = fl_mock_view_keyboard_send_key_event;
iface->text_filter_key_press = fl_mock_view_keyboard_text_filter_key_press;
iface->get_messenger = fl_mock_view_keyboard_get_messenger;
iface->redispatch_event = fl_mock_view_keyboard_redispatch_event;
iface->lookup_key = fl_mock_view_keyboard_lookup_key;
}
Expand Down Expand Up @@ -410,7 +403,8 @@ class KeyboardTester {
respondToTextInputWith(false);
setLayout(kLayoutUs);

handler_ = fl_keyboard_manager_new(FL_KEYBOARD_VIEW_DELEGATE(view_));
handler_ = fl_keyboard_manager_new(FL_BINARY_MESSENGER(view_->messenger),
FL_KEYBOARD_VIEW_DELEGATE(view_));
}

~KeyboardTester() {
Expand Down
7 changes: 0 additions & 7 deletions shell/platform/linux/fl_keyboard_view_delegate.cc
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,6 @@ gboolean fl_keyboard_view_delegate_text_filter_key_press(
self, event);
}

FlBinaryMessenger* fl_keyboard_view_delegate_get_messenger(
FlKeyboardViewDelegate* self) {
g_return_val_if_fail(FL_IS_KEYBOARD_VIEW_DELEGATE(self), nullptr);

return FL_KEYBOARD_VIEW_DELEGATE_GET_IFACE(self)->get_messenger(self);
}

void fl_keyboard_view_delegate_redispatch_event(FlKeyboardViewDelegate* self,
FlKeyEvent* event) {
g_return_if_fail(FL_IS_KEYBOARD_VIEW_DELEGATE(self));
Expand Down
13 changes: 0 additions & 13 deletions shell/platform/linux/fl_keyboard_view_delegate.h
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,6 @@ struct _FlKeyboardViewDelegateInterface {
gboolean (*text_filter_key_press)(FlKeyboardViewDelegate* delegate,
FlKeyEvent* event);

FlBinaryMessenger* (*get_messenger)(FlKeyboardViewDelegate* delegate);

void (*redispatch_event)(FlKeyboardViewDelegate* delegate, FlKeyEvent* event);

guint (*lookup_key)(FlKeyboardViewDelegate* view_delegate,
Expand Down Expand Up @@ -81,17 +79,6 @@ gboolean fl_keyboard_view_delegate_text_filter_key_press(
FlKeyboardViewDelegate* delegate,
FlKeyEvent* event);

/**
* fl_keyboard_view_delegate_get_messenger:
*
* Returns a binary messenger that can be used to send messages to the
* framework.
*
* The ownership of messenger is kept by the view delegate.
*/
FlBinaryMessenger* fl_keyboard_view_delegate_get_messenger(
FlKeyboardViewDelegate* delegate);

/**
* fl_keyboard_view_delegate_redispatch_event:
*
Expand Down
7 changes: 1 addition & 6 deletions shell/platform/linux/fl_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ static void init_keyboard(FlView* self) {
fl_keyboard_handler_new(messenger, FL_KEYBOARD_VIEW_DELEGATE(self));
g_clear_object(&self->keyboard_manager);
self->keyboard_manager =
fl_keyboard_manager_new(FL_KEYBOARD_VIEW_DELEGATE(self));
fl_keyboard_manager_new(messenger, FL_KEYBOARD_VIEW_DELEGATE(self));
}

static void init_scrolling(FlView* self) {
Expand Down Expand Up @@ -385,11 +385,6 @@ static void fl_view_keyboard_delegate_iface_init(
event);
};

iface->get_messenger = [](FlKeyboardViewDelegate* view_delegate) {
FlView* self = FL_VIEW(view_delegate);
return fl_engine_get_binary_messenger(self->engine);
};

iface->redispatch_event = [](FlKeyboardViewDelegate* view_delegate,
FlKeyEvent* event) {
GdkEventType event_type =
Expand Down