mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-09-30 00:31:14 +00:00
Assistant: Make Result bitmaps virtual/lazy
Result classes now return their bitmap via a virtual Gfx::Bitmap* getter. This effectively makes bitmap fetching lazier, since only results that end up on screen actually get asked for their bitmap. This drastically reduces the amount of work done by the FileProvider background worker.
This commit is contained in:
parent
1c29e2f3f7
commit
513e67e2eb
Notes:
sideshowbarker
2024-07-18 11:03:05 +09:00
Author: https://github.com/awesomekling Commit: https://github.com/SerenityOS/serenity/commit/513e67e2ebb
|
@ -112,6 +112,11 @@ void CalculatorProvider::query(String const& query, Function<void(NonnullRefPtrV
|
|||
on_complete(results);
|
||||
}
|
||||
|
||||
Gfx::Bitmap const* FileResult::bitmap() const
|
||||
{
|
||||
return GUI::Icon::default_icon("filetype-folder").bitmap_for_size(16);
|
||||
}
|
||||
|
||||
void FileProvider::query(const String& query, Function<void(NonnullRefPtrVector<Result>)> on_complete)
|
||||
{
|
||||
build_filesystem_cache();
|
||||
|
|
|
@ -24,7 +24,8 @@ public:
|
|||
|
||||
virtual void activate() const = 0;
|
||||
|
||||
RefPtr<Gfx::Bitmap> bitmap() { return m_bitmap; }
|
||||
virtual Gfx::Bitmap const* bitmap() const { return nullptr; }
|
||||
|
||||
String const& title() const { return m_title; }
|
||||
String const& subtitle() const { return m_subtitle; }
|
||||
int score() const { return m_score; }
|
||||
|
@ -36,73 +37,93 @@ public:
|
|||
}
|
||||
|
||||
protected:
|
||||
Result(RefPtr<Gfx::Bitmap> bitmap, String title, String subtitle, int score = 0)
|
||||
: m_bitmap(move(bitmap))
|
||||
, m_title(move(title))
|
||||
Result(String title, String subtitle, int score = 0)
|
||||
: m_title(move(title))
|
||||
, m_subtitle(move(subtitle))
|
||||
, m_score(score)
|
||||
{
|
||||
}
|
||||
|
||||
private:
|
||||
RefPtr<Gfx::Bitmap> m_bitmap;
|
||||
String m_title;
|
||||
String m_subtitle;
|
||||
int m_score { 0 };
|
||||
};
|
||||
|
||||
class AppResult : public Result {
|
||||
class AppResult final : public Result {
|
||||
public:
|
||||
AppResult(RefPtr<Gfx::Bitmap> bitmap, String title, String subtitle, NonnullRefPtr<Desktop::AppFile> af, int score)
|
||||
: Result(move(bitmap), move(title), move(subtitle), score)
|
||||
: Result(move(title), move(subtitle), score)
|
||||
, m_app_file(move(af))
|
||||
, m_bitmap(move(bitmap))
|
||||
{
|
||||
}
|
||||
~AppResult() override = default;
|
||||
void activate() const override;
|
||||
|
||||
virtual Gfx::Bitmap const* bitmap() const override { return m_bitmap; }
|
||||
|
||||
private:
|
||||
NonnullRefPtr<Desktop::AppFile> m_app_file;
|
||||
RefPtr<Gfx::Bitmap> m_bitmap;
|
||||
};
|
||||
|
||||
class CalculatorResult : public Result {
|
||||
public:
|
||||
explicit CalculatorResult(String title)
|
||||
: Result(GUI::Icon::default_icon("app-calculator").bitmap_for_size(16), move(title), "'Enter' will copy to clipboard"sv, 100)
|
||||
: Result(move(title), "'Enter' will copy to clipboard"sv, 100)
|
||||
, m_bitmap(GUI::Icon::default_icon("app-calculator").bitmap_for_size(16))
|
||||
{
|
||||
}
|
||||
~CalculatorResult() override = default;
|
||||
void activate() const override;
|
||||
|
||||
virtual Gfx::Bitmap const* bitmap() const override { return m_bitmap; }
|
||||
|
||||
private:
|
||||
RefPtr<Gfx::Bitmap> m_bitmap;
|
||||
};
|
||||
|
||||
class FileResult : public Result {
|
||||
public:
|
||||
explicit FileResult(String title, int score)
|
||||
: Result(GUI::Icon::default_icon("filetype-folder").bitmap_for_size(16), move(title), "", score)
|
||||
: Result(move(title), "", score)
|
||||
{
|
||||
}
|
||||
~FileResult() override = default;
|
||||
void activate() const override;
|
||||
|
||||
virtual Gfx::Bitmap const* bitmap() const override;
|
||||
};
|
||||
|
||||
class TerminalResult : public Result {
|
||||
public:
|
||||
explicit TerminalResult(String command)
|
||||
: Result(GUI::Icon::default_icon("app-terminal").bitmap_for_size(16), move(command), "Run command in Terminal"sv, 100)
|
||||
: Result(move(command), "Run command in Terminal"sv, 100)
|
||||
, m_bitmap(GUI::Icon::default_icon("app-terminal").bitmap_for_size(16))
|
||||
{
|
||||
}
|
||||
~TerminalResult() override = default;
|
||||
void activate() const override;
|
||||
|
||||
private:
|
||||
RefPtr<Gfx::Bitmap> m_bitmap;
|
||||
};
|
||||
|
||||
class URLResult : public Result {
|
||||
public:
|
||||
explicit URLResult(const URL& url)
|
||||
: Result(GUI::Icon::default_icon("app-browser").bitmap_for_size(16), url.to_string(), "'Enter' will open this URL in the browser"sv, 50)
|
||||
: Result(url.to_string(), "'Enter' will open this URL in the browser"sv, 50)
|
||||
, m_bitmap(GUI::Icon::default_icon("app-browser").bitmap_for_size(16))
|
||||
{
|
||||
}
|
||||
~URLResult() override = default;
|
||||
void activate() const override;
|
||||
|
||||
virtual Gfx::Bitmap const* bitmap() const override { return m_bitmap; }
|
||||
|
||||
private:
|
||||
RefPtr<Gfx::Bitmap> m_bitmap;
|
||||
};
|
||||
|
||||
class Provider {
|
||||
|
|
Loading…
Reference in a new issue