diff options
author | Andreas Kling <kling@serenityos.org> | 2020-11-19 22:21:16 +0100 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2020-11-19 23:10:21 +0100 |
commit | 10b534849dcb07fbaa600e5c5b58ca784ced7737 (patch) | |
tree | 24f4aa709e159ffdfb52731a9a597702db357762 | |
parent | 6a569bbdd76a23d0875fa06c10e4ddb5838b0259 (diff) | |
download | serenity-10b534849dcb07fbaa600e5c5b58ca784ced7737.zip |
LibWeb: Remove ancient HTML_DEBUG debug logging
-rw-r--r-- | Libraries/LibWeb/CSS/StyleProperties.cpp | 6 | ||||
-rw-r--r-- | Libraries/LibWeb/CSS/StyleResolver.cpp | 7 | ||||
-rw-r--r-- | Libraries/LibWeb/Dump.h | 2 | ||||
-rw-r--r-- | Libraries/LibWeb/InProcessWebView.cpp | 5 | ||||
-rw-r--r-- | Libraries/LibWeb/Layout/LayoutBlock.cpp | 4 | ||||
-rw-r--r-- | Libraries/LibWeb/Page/EventHandler.cpp | 6 |
6 files changed, 1 insertions, 29 deletions
diff --git a/Libraries/LibWeb/CSS/StyleProperties.cpp b/Libraries/LibWeb/CSS/StyleProperties.cpp index 3198e5c87f..c0b40984b3 100644 --- a/Libraries/LibWeb/CSS/StyleProperties.cpp +++ b/Libraries/LibWeb/CSS/StyleProperties.cpp @@ -159,17 +159,11 @@ void StyleProperties::load_font() const if (file_name.is_null()) continue; -#ifdef HTML_DEBUG - dbg() << "Found font " << file_name << " for family " << font_family << " weight " << font_weight; -#endif m_font = Gfx::Font::load_from_file(String::format("/res/fonts/%s", file_name.characters())); FontCache::the().set({ font_name, font_weight }, *m_font); return; } -#ifdef HTML_DEBUG - dbg() << "Failed to find a font for family " << font_family << " weight " << font_weight; -#endif if (font_weight == "bold") m_font = Gfx::Font::default_bold_font(); else diff --git a/Libraries/LibWeb/CSS/StyleResolver.cpp b/Libraries/LibWeb/CSS/StyleResolver.cpp index 96627a6cc4..accc16032f 100644 --- a/Libraries/LibWeb/CSS/StyleResolver.cpp +++ b/Libraries/LibWeb/CSS/StyleResolver.cpp @@ -100,13 +100,6 @@ Vector<MatchingRule> StyleResolver::collect_matching_rules(const DOM::Element& e ++style_sheet_index; }); -#ifdef HTML_DEBUG - dbgprintf("Rules matching Element{%p}\n", &element); - for (auto& rule : matching_rules) { - dump_rule(rule); - } -#endif - return matching_rules; } diff --git a/Libraries/LibWeb/Dump.h b/Libraries/LibWeb/Dump.h index 5f7512371c..6bb1594837 100644 --- a/Libraries/LibWeb/Dump.h +++ b/Libraries/LibWeb/Dump.h @@ -36,6 +36,4 @@ void dump_sheet(const CSS::StyleSheet&); void dump_rule(const CSS::StyleRule&); void dump_selector(const CSS::Selector&); -#undef HTML_DEBUG - } diff --git a/Libraries/LibWeb/InProcessWebView.cpp b/Libraries/LibWeb/InProcessWebView.cpp index 15dd3eb155..2580f80b4e 100644 --- a/Libraries/LibWeb/InProcessWebView.cpp +++ b/Libraries/LibWeb/InProcessWebView.cpp @@ -249,11 +249,6 @@ void InProcessWebView::layout_and_sync_size() } page().main_frame().set_viewport_rect(viewport_rect_in_content_coordinates()); - -#ifdef HTML_DEBUG - dbgprintf("\033[33;1mLayout tree after layout:\033[0m\n"); - ::dump_tree(*layout_root()); -#endif } void InProcessWebView::resize_event(GUI::ResizeEvent& event) diff --git a/Libraries/LibWeb/Layout/LayoutBlock.cpp b/Libraries/LibWeb/Layout/LayoutBlock.cpp index 073bc24267..d0acaa4e89 100644 --- a/Libraries/LibWeb/Layout/LayoutBlock.cpp +++ b/Libraries/LibWeb/Layout/LayoutBlock.cpp @@ -450,10 +450,6 @@ void LayoutBlock::compute_width() total_px += value.to_px(*this); } -#ifdef HTML_DEBUG - dbg() << "Total: " << total_px; -#endif - if (!is_replaced() && !is_inline()) { // 10.3.3 Block-level, non-replaced elements in normal flow // If 'width' is not 'auto' and 'border-left-width' + 'padding-left' + 'width' + 'padding-right' + 'border-right-width' (plus any of 'margin-left' or 'margin-right' that are not 'auto') is larger than the width of the containing block, then any 'auto' values for 'margin-left' or 'margin-right' are, for the following rules, treated as zero. diff --git a/Libraries/LibWeb/Page/EventHandler.cpp b/Libraries/LibWeb/Page/EventHandler.cpp index ad10e0b77d..30bda6c150 100644 --- a/Libraries/LibWeb/Page/EventHandler.cpp +++ b/Libraries/LibWeb/Page/EventHandler.cpp @@ -250,12 +250,8 @@ bool EventHandler::handle_mousemove(const Gfx::IntPoint& position, unsigned butt if (node->is_text()) is_hovering_text = true; hovered_link_element = node->enclosing_link_element(); - if (hovered_link_element) { -#ifdef HTML_DEBUG - dbg() << "InProcessWebView: hovering over a link to " << hovered_link_element->href(); -#endif + if (hovered_link_element) is_hovering_link = true; - } auto offset = compute_mouse_event_offset(position, *result.layout_node); node->dispatch_event(UIEvents::MouseEvent::create("mousemove", offset.x(), offset.y())); if (!layout_root()) |