aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--components/layout_2020/display_list/items.rs17
-rw-r--r--components/layout_2020/display_list/mod.rs6
-rw-r--r--components/layout_2020/display_list/webrender_helpers.rs33
-rw-r--r--components/layout_2020/fragment.rs2
-rw-r--r--components/layout_2020/lib.rs4
-rw-r--r--components/layout_2020/opaque_node.rs2
-rw-r--r--components/layout_2020/query.rs9
-rw-r--r--components/layout_thread_2020/lib.rs22
-rw-r--r--tests/unit/metrics/paint_time.rs32
9 files changed, 22 insertions, 105 deletions
diff --git a/components/layout_2020/display_list/items.rs b/components/layout_2020/display_list/items.rs
deleted file mode 100644
index 8797480329d..00000000000
--- a/components/layout_2020/display_list/items.rs
+++ /dev/null
@@ -1,17 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
-
-use euclid::Vector2D;
-use std::collections::HashMap;
-use std::f32;
-use webrender_api::units::LayoutPixel;
-use webrender_api::ExternalScrollId;
-
-pub use style::dom::OpaqueNode;
-
-#[derive(Serialize)]
-pub struct DisplayList {}
-
-/// The type of the scroll offset list. This is only populated if WebRender is in use.
-pub type ScrollOffsetMap = HashMap<ExternalScrollId, Vector2D<f32, LayoutPixel>>;
diff --git a/components/layout_2020/display_list/mod.rs b/components/layout_2020/display_list/mod.rs
deleted file mode 100644
index 88afb0ed6a6..00000000000
--- a/components/layout_2020/display_list/mod.rs
+++ /dev/null
@@ -1,6 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
-
-pub mod items;
-mod webrender_helpers;
diff --git a/components/layout_2020/display_list/webrender_helpers.rs b/components/layout_2020/display_list/webrender_helpers.rs
deleted file mode 100644
index 951a0d61f95..00000000000
--- a/components/layout_2020/display_list/webrender_helpers.rs
+++ /dev/null
@@ -1,33 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
-
-use crate::display_list::items::DisplayList;
-use msg::constellation_msg::PipelineId;
-use webrender_api::units::LayoutSize;
-use webrender_api::{self, DisplayListBuilder};
-
-/// Contentful paint, for the purpose of
-/// https://w3c.github.io/paint-timing/#first-contentful-paint
-/// (i.e. the display list contains items of type text,
-/// image, non-white canvas or SVG). Used by metrics.
-pub struct IsContentful(pub bool);
-
-impl DisplayList {
- pub fn convert_to_webrender(
- &mut self,
- pipeline_id: PipelineId,
- ) -> (DisplayListBuilder, IsContentful) {
- let webrender_pipeline = pipeline_id.to_webrender();
-
- let builder = DisplayListBuilder::with_capacity(
- webrender_pipeline,
- LayoutSize::zero(),
- 1024 * 1024, // 1 MB of space
- );
-
- let is_contentful = IsContentful(false);
-
- (builder, is_contentful)
- }
-}
diff --git a/components/layout_2020/fragment.rs b/components/layout_2020/fragment.rs
index f1fa89d141d..71e23b7ffaa 100644
--- a/components/layout_2020/fragment.rs
+++ b/components/layout_2020/fragment.rs
@@ -5,11 +5,11 @@
//! The `Fragment` type, which represents the leaves of the layout tree.
use crate::context::LayoutContext;
-use crate::display_list::items::OpaqueNode;
use crate::ServoArc;
use app_units::Au;
use script_layout_interface::wrapper_traits::{PseudoElementType, ThreadSafeLayoutNode};
use serde::ser::{Serialize, SerializeStruct, Serializer};
+use style::dom::OpaqueNode;
use style::logical_geometry::{LogicalMargin, LogicalRect};
use style::properties::ComputedValues;
use style::selector_parser::RestyleDamage;
diff --git a/components/layout_2020/lib.rs b/components/layout_2020/lib.rs
index e08d962df22..4af20789529 100644
--- a/components/layout_2020/lib.rs
+++ b/components/layout_2020/lib.rs
@@ -4,12 +4,8 @@
#![deny(unsafe_code)]
-#[macro_use]
-extern crate serde;
-
pub mod context;
pub mod data;
-pub mod display_list;
mod fragment;
pub mod opaque_node;
pub mod query;
diff --git a/components/layout_2020/opaque_node.rs b/components/layout_2020/opaque_node.rs
index 3d56ef21dae..36df7c640f4 100644
--- a/components/layout_2020/opaque_node.rs
+++ b/components/layout_2020/opaque_node.rs
@@ -2,9 +2,9 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at https://mozilla.org/MPL/2.0/. */
-use crate::display_list::items::OpaqueNode;
use libc::c_void;
use script_traits::UntrustedNodeAddress;
+use style::dom::OpaqueNode;
pub trait OpaqueNodeMethods {
fn to_untrusted_node_address(&self) -> UntrustedNodeAddress;
diff --git a/components/layout_2020/query.rs b/components/layout_2020/query.rs
index fc308005806..23816521c70 100644
--- a/components/layout_2020/query.rs
+++ b/components/layout_2020/query.rs
@@ -5,9 +5,9 @@
//! Utilities for querying the layout, as needed by the layout thread.
use crate::context::LayoutContext;
-use crate::display_list::items::{DisplayList, OpaqueNode, ScrollOffsetMap};
use app_units::Au;
use euclid::default::{Point2D, Rect};
+use euclid::Vector2D;
use ipc_channel::ipc::IpcSender;
use msg::constellation_msg::PipelineId;
use script_layout_interface::rpc::TextIndexResponse;
@@ -17,9 +17,12 @@ use script_layout_interface::rpc::{OffsetParentResponse, ResolvedStyleResponse,
use script_layout_interface::wrapper_traits::{LayoutNode, ThreadSafeLayoutNode};
use script_traits::LayoutMsg as ConstellationMsg;
use script_traits::UntrustedNodeAddress;
+use std::collections::HashMap;
use std::sync::{Arc, Mutex};
+use style::dom::OpaqueNode;
use style::properties::PropertyId;
use style::selector_parser::PseudoElement;
+use webrender_api::units::LayoutPixel;
use webrender_api::ExternalScrollId;
/// Mutable data belonging to the LayoutThread.
@@ -30,7 +33,7 @@ pub struct LayoutThreadData {
pub constellation_chan: IpcSender<ConstellationMsg>,
/// The root stacking context.
- pub display_list: Option<DisplayList>,
+ pub display_list: Option<webrender_api::DisplayListBuilder>,
/// A queued response for the union of the content boxes of a node.
pub content_box_response: Option<Rect<Au>>,
@@ -57,7 +60,7 @@ pub struct LayoutThreadData {
pub style_response: StyleResponse,
/// Scroll offsets of scrolling regions.
- pub scroll_offsets: ScrollOffsetMap,
+ pub scroll_offsets: HashMap<ExternalScrollId, Vector2D<f32, LayoutPixel>>,
/// Index in a text fragment. We need this do determine the insertion point.
pub text_index_response: TextIndexResponse,
diff --git a/components/layout_thread_2020/lib.rs b/components/layout_thread_2020/lib.rs
index 9c3e03c5ee0..ec31d858b9f 100644
--- a/components/layout_thread_2020/lib.rs
+++ b/components/layout_thread_2020/lib.rs
@@ -37,7 +37,6 @@ use histogram::Histogram;
use ipc_channel::ipc::{self, IpcReceiver, IpcSender};
use ipc_channel::router::ROUTER;
use layout::context::LayoutContext;
-use layout::display_list::items::DisplayList;
use layout::query::{
process_content_box_request, process_content_boxes_request, LayoutRPCImpl, LayoutThreadData,
};
@@ -1337,36 +1336,35 @@ impl LayoutThread {
document.will_paint();
}
- let mut display_list = DisplayList {};
-
- debug!("Layout done!");
-
- // TODO: Avoid the temporary conversion and build webrender sc/dl directly!
- let (builder, is_contentful) = display_list.convert_to_webrender(self.id);
-
let viewport_size = Size2D::new(
self.viewport_size.width.to_f32_px(),
self.viewport_size.height.to_f32_px(),
);
+ let viewport_size = webrender_api::units::LayoutSize::from_untyped(viewport_size);
+
+ let display_list =
+ webrender_api::DisplayListBuilder::new(self.id.to_webrender(), viewport_size);
+ let is_contentful = false;
+
+ debug!("Layout done!");
+
let mut epoch = self.epoch.get();
epoch.next();
self.epoch.set(epoch);
- let viewport_size = webrender_api::units::LayoutSize::from_untyped(viewport_size);
-
// Observe notifications about rendered frames if needed right before
// sending the display list to WebRender in order to set time related
// Progressive Web Metrics.
self.paint_time_metrics
- .maybe_observe_paint_time(self, epoch, is_contentful.0);
+ .maybe_observe_paint_time(self, epoch, is_contentful);
let mut txn = webrender_api::Transaction::new();
txn.set_display_list(
webrender_api::Epoch(epoch.0),
None,
viewport_size,
- builder.finalize(),
+ display_list.finalize(),
true,
);
txn.generate_frame();
diff --git a/tests/unit/metrics/paint_time.rs b/tests/unit/metrics/paint_time.rs
index c404e4d6e7b..ac8ade3842c 100644
--- a/tests/unit/metrics/paint_time.rs
+++ b/tests/unit/metrics/paint_time.rs
@@ -4,13 +4,10 @@
use gfx_traits::Epoch;
use ipc_channel::ipc;
-use layout::display_list::items::{BaseDisplayItem, CommonDisplayItem, DisplayItem, DisplayList};
use metrics::{PaintTimeMetrics, ProfilerMetadataFactory, ProgressiveWebMetric};
use msg::constellation_msg::TEST_PIPELINE_ID;
use profile_traits::time::{ProfilerChan, TimerMetadata};
use servo_url::ServoUrl;
-use webrender_api::units::{LayoutRect, LayoutSize};
-use webrender_api::{AlphaType, ColorF, ImageDisplayItem, ImageKey, ImageRendering};
struct DummyProfilerMetadataFactory {}
impl ProfilerMetadataFactory for DummyProfilerMetadataFactory {
@@ -49,7 +46,7 @@ fn test_paint_metrics_construction() {
);
}
-fn test_common(display_list: &DisplayList, epoch: Epoch) -> PaintTimeMetrics {
+fn test_common(display_list_is_contentful: bool, epoch: Epoch) -> PaintTimeMetrics {
let (sender, _) = ipc::channel().unwrap();
let profiler_chan = ProfilerChan(sender);
let (layout_sender, _) = ipc::channel().unwrap();
@@ -66,7 +63,7 @@ fn test_common(display_list: &DisplayList, epoch: Epoch) -> PaintTimeMetrics {
paint_time_metrics.maybe_observe_paint_time(
&dummy_profiler_metadata_factory,
epoch,
- &*display_list,
+ display_list_is_contentful,
);
// Should not set any metric until navigation start is set.
@@ -95,12 +92,8 @@ fn test_common(display_list: &DisplayList, epoch: Epoch) -> PaintTimeMetrics {
#[test]
fn test_first_paint_setter() {
- let empty_display_list = DisplayList {
- list: Vec::new(),
- clip_scroll_nodes: Vec::new(),
- };
let epoch = Epoch(0);
- let paint_time_metrics = test_common(&empty_display_list, epoch);
+ let paint_time_metrics = test_common(false, epoch);
let now = time::precise_time_ns();
paint_time_metrics.maybe_set_metric(epoch, now);
assert!(
@@ -116,25 +109,8 @@ fn test_first_paint_setter() {
#[test]
fn test_first_contentful_paint_setter() {
- let image = DisplayItem::Image(CommonDisplayItem::new(
- BaseDisplayItem::empty(),
- ImageDisplayItem {
- bounds: LayoutRect::zero(),
- common: layout::display_list::items::empty_common_item_properties(),
- image_key: ImageKey::DUMMY,
- stretch_size: LayoutSize::zero(),
- tile_spacing: LayoutSize::zero(),
- image_rendering: ImageRendering::Auto,
- alpha_type: AlphaType::PremultipliedAlpha,
- color: ColorF::WHITE,
- },
- ));
- let display_list = DisplayList {
- list: vec![image],
- clip_scroll_nodes: Vec::new(),
- };
let epoch = Epoch(0);
- let paint_time_metrics = test_common(&display_list, epoch);
+ let paint_time_metrics = test_common(true, epoch);
let now = time::precise_time_ns();
paint_time_metrics.maybe_set_metric(epoch, now);
assert!(
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-27 20:46:02 +0000