Browse Source

LibWeb+WebContent+headless-browser: Remove PaintContext::scroll_offset()

Nobody uses this. They get the scroll_offset from the BlockContainer
instead.
Sam Atkins 2 years ago
parent
commit
d3476c28ba

+ 1 - 2
Userland/Libraries/LibWeb/Painting/PaintContext.cpp

@@ -8,10 +8,9 @@
 
 
 namespace Web {
 namespace Web {
 
 
-PaintContext::PaintContext(Gfx::Painter& painter, Palette const& palette, Gfx::IntPoint scroll_offset)
+PaintContext::PaintContext(Gfx::Painter& painter, Palette const& palette)
     : m_painter(painter)
     : m_painter(painter)
     , m_palette(palette)
     , m_palette(palette)
-    , m_scroll_offset(scroll_offset)
 {
 {
 }
 }
 
 

+ 2 - 5
Userland/Libraries/LibWeb/Painting/PaintContext.h

@@ -16,7 +16,7 @@ namespace Web {
 
 
 class PaintContext {
 class PaintContext {
 public:
 public:
-    PaintContext(Gfx::Painter& painter, Palette const& palette, Gfx::IntPoint scroll_offset);
+    PaintContext(Gfx::Painter& painter, Palette const& palette);
 
 
     Gfx::Painter& painter() const { return m_painter; }
     Gfx::Painter& painter() const { return m_painter; }
     Palette const& palette() const { return m_palette; }
     Palette const& palette() const { return m_palette; }
@@ -32,14 +32,12 @@ public:
     Gfx::IntRect viewport_rect() const { return m_viewport_rect; }
     Gfx::IntRect viewport_rect() const { return m_viewport_rect; }
     void set_viewport_rect(Gfx::IntRect const& rect) { m_viewport_rect = rect; }
     void set_viewport_rect(Gfx::IntRect const& rect) { m_viewport_rect = rect; }
 
 
-    Gfx::IntPoint scroll_offset() const { return m_scroll_offset; }
-
     bool has_focus() const { return m_focus; }
     bool has_focus() const { return m_focus; }
     void set_has_focus(bool focus) { m_focus = focus; }
     void set_has_focus(bool focus) { m_focus = focus; }
 
 
     PaintContext clone(Gfx::Painter& painter) const
     PaintContext clone(Gfx::Painter& painter) const
     {
     {
-        auto clone = PaintContext(painter, m_palette, m_scroll_offset);
+        auto clone = PaintContext(painter, m_palette);
         clone.m_viewport_rect = m_viewport_rect;
         clone.m_viewport_rect = m_viewport_rect;
         clone.m_should_show_line_box_borders = m_should_show_line_box_borders;
         clone.m_should_show_line_box_borders = m_should_show_line_box_borders;
         clone.m_focus = m_focus;
         clone.m_focus = m_focus;
@@ -52,7 +50,6 @@ private:
     Palette m_palette;
     Palette m_palette;
     Optional<SVGContext> m_svg_context;
     Optional<SVGContext> m_svg_context;
     Gfx::IntRect m_viewport_rect;
     Gfx::IntRect m_viewport_rect;
-    Gfx::IntPoint m_scroll_offset;
     bool m_should_show_line_box_borders { false };
     bool m_should_show_line_box_borders { false };
     bool m_focus { false };
     bool m_focus { false };
 };
 };

+ 1 - 1
Userland/Services/WebContent/PageHost.cpp

@@ -118,7 +118,7 @@ void PageHost::paint(Web::DevicePixelRect const& content_rect, Gfx::Bitmap& targ
         return;
         return;
     }
     }
 
 
-    Web::PaintContext context(painter, palette(), content_rect.top_left().to_type<int>());
+    Web::PaintContext context(painter, palette());
     context.set_should_show_line_box_borders(m_should_show_line_box_borders);
     context.set_should_show_line_box_borders(m_should_show_line_box_borders);
     context.set_viewport_rect(content_rect.to_type<int>());
     context.set_viewport_rect(content_rect.to_type<int>());
     context.set_has_focus(m_has_focus);
     context.set_has_focus(m_has_focus);

+ 1 - 1
Userland/Utilities/headless-browser.cpp

@@ -87,7 +87,7 @@ public:
             return;
             return;
         }
         }
 
 
-        Web::PaintContext context(painter, palette(), content_rect.top_left().to_type<int>());
+        Web::PaintContext context(painter, palette());
         context.set_should_show_line_box_borders(false);
         context.set_should_show_line_box_borders(false);
         context.set_viewport_rect(content_rect.to_type<int>());
         context.set_viewport_rect(content_rect.to_type<int>());
         context.set_has_focus(true);
         context.set_has_focus(true);