diff --git a/Libraries/LibWebView/Application.cpp b/Libraries/LibWebView/Application.cpp index faed92da45b..fb3d39c194c 100644 --- a/Libraries/LibWebView/Application.cpp +++ b/Libraries/LibWebView/Application.cpp @@ -321,16 +321,6 @@ Optional Application::find_process(pid_t pid) return m_process_manager.find_process(pid); } -void Application::update_process_statistics() -{ - m_process_manager.update_all_process_statistics(); -} - -String Application::generate_process_statistics_html() -{ - return m_process_manager.generate_html(); -} - void Application::send_updated_process_statistics_to_view(ViewImplementation& view) { m_process_manager.update_all_process_statistics(); diff --git a/Libraries/LibWebView/Application.h b/Libraries/LibWebView/Application.h index 4d46b1d6a80..055c78669c5 100644 --- a/Libraries/LibWebView/Application.h +++ b/Libraries/LibWebView/Application.h @@ -56,10 +56,6 @@ public: #endif Optional find_process(pid_t); - // FIXME: Should we just expose the ProcessManager via a getter? - void update_process_statistics(); - String generate_process_statistics_html(); - void send_updated_process_statistics_to_view(ViewImplementation&); ErrorOr path_for_downloaded_file(StringView file) const; diff --git a/Libraries/LibWebView/ProcessManager.cpp b/Libraries/LibWebView/ProcessManager.cpp index e3a0352baf3..8114cba2fd1 100644 --- a/Libraries/LibWebView/ProcessManager.cpp +++ b/Libraries/LibWebView/ProcessManager.cpp @@ -6,7 +6,6 @@ #include #include -#include #include #include #include @@ -121,89 +120,6 @@ void ProcessManager::update_all_process_statistics() (void)update_process_statistics(m_statistics); } -String ProcessManager::generate_html() -{ - Threading::MutexLocker locker { m_lock }; - StringBuilder builder; - - builder.append(R"( - - - Task Manager - - - - - - - - - - - - - - )"sv); - - m_statistics.for_each_process([&](auto const& process) { - builder.append(""sv); - builder.append(""sv); - builder.append(""sv); - builder.append(""sv); - builder.append(""sv); - builder.append(""sv); - }); - - builder.append(R"( - -
NamePIDMemory UsageCPU %
"sv); - auto& process_handle = this->find_process(process.pid).value(); - builder.append(WebView::process_name_from_type(process_handle.type())); - if (process_handle.title().has_value()) - builder.appendff(" - {}", escape_html_entities(*process_handle.title())); - builder.append(""sv); - builder.append(String::number(process.pid)); - builder.append(""sv); - builder.append(human_readable_size(process.memory_usage_bytes)); - builder.append(""sv); - builder.append(MUST(String::formatted("{:.1f}", process.cpu_percent))); - builder.append("
- - - )"sv); - - return builder.to_string_without_validation(); -} - String ProcessManager::serialize_json() { Threading::MutexLocker locker { m_lock }; diff --git a/Libraries/LibWebView/ProcessManager.h b/Libraries/LibWebView/ProcessManager.h index 478b9f56341..f501718f129 100644 --- a/Libraries/LibWebView/ProcessManager.h +++ b/Libraries/LibWebView/ProcessManager.h @@ -7,8 +7,6 @@ #pragma once #include -#include -#include #include #include #include @@ -36,7 +34,6 @@ public: #endif void update_all_process_statistics(); - String generate_html(); String serialize_json(); Function on_process_exited;