mirror of
https://github.com/servo/servo.git
synced 2025-08-03 20:50:07 +01:00
Auto merge of #23007 - BartGitHub:remove-pipeline-visibility-checks, r=jdm
Remove pipeline visibility checks <!-- Please describe your changes on the following line: --> This includes the following changes/clean-ups: - Remove ```is_visible``` field from ```Pipeline``` struct - Remove ```SetVisible``` script message, and related message sending/handling. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #22986 <!-- Either: --> - [x] These changes do not require tests because the issue description states having a passing build is enough for a pull request. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23007) <!-- Reviewable:end -->
This commit is contained in:
commit
c12e341c0c
3 changed files with 0 additions and 48 deletions
|
@ -175,8 +175,6 @@ pub enum ScriptMsg {
|
|||
/// Notification that this iframe should be removed.
|
||||
/// Returns a list of pipelines which were closed.
|
||||
RemoveIFrame(BrowsingContextId, IpcSender<Vec<PipelineId>>),
|
||||
/// Change pipeline visibility
|
||||
SetVisible(bool),
|
||||
/// Notifies constellation that an iframe's visibility has been changed.
|
||||
VisibilityChangeComplete(bool),
|
||||
/// A load has been requested in an IFrame.
|
||||
|
@ -243,7 +241,6 @@ impl fmt::Debug for ScriptMsg {
|
|||
ReplaceHistoryState(..) => "ReplaceHistoryState",
|
||||
JointSessionHistoryLength(..) => "JointSessionHistoryLength",
|
||||
RemoveIFrame(..) => "RemoveIFrame",
|
||||
SetVisible(..) => "SetVisible",
|
||||
VisibilityChangeComplete(..) => "VisibilityChangeComplete",
|
||||
ScriptLoadedURLInIFrame(..) => "ScriptLoadedURLInIFrame",
|
||||
ScriptNewIFrame(..) => "ScriptNewIFrame",
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue