New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
script_layout: Remove script to layout messages #32081
script_layout: Remove script to layout messages #32081
Conversation
🔨 Triggering try run (#8690595859) for Linux WPT, MacOS, Windows, Android |
Test results for linux-wpt-layout-2020 from try job (#8690595859): Flaky unexpected result (18)
Stable unexpected results that are known to be intermittent (12)
|
Test results for linux-wpt-layout-2013 from try job (#8690595859): Flaky unexpected result (18)
Stable unexpected results that are known to be intermittent (9)
|
|
7de4bb1
to
e8d7d0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
@@ -466,7 +466,7 @@ pub enum InputMethodType { | |||
} | |||
|
|||
#[derive(Clone, Copy, Debug, Deserialize, Serialize)] | |||
/// The equivalent of script_layout_interface::message::Msg | |||
/// The equivalent of script_layout_interface::Msg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Msg
is removed in this patch, so wouldn't this comment be incorrect? I guess this enum is now unused and can be deleted along with the HangAnnotation::Layout
variant. I'm fine if you think that can happen in a separate patch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. I wasn't sure if these would be used again someday, but I think we should remove them now because they are dead code.
Thanks so much for the review!
Instead of communicating with layout via messages, script can simply call methods on the layout trait. This simplifies the way that script communicates with layout and opens the path to sharing more data structures between the two systems.
e8d7d0e
to
675df5d
Compare
Instead of communicating with layout via messages, script can simply
call methods on the layout trait. This simplifies the way that script
communicates with layout and opens the path to sharing more data
structures between the two systems. This is part of a continuing
series of cleanups after removing the layout thread.
./mach build -d
does not report any errors./mach test-tidy
does not report any errors