Simplifies clip-indicating member

brave · Jan 10, 2025 · f6f8e65 · f6f8e65
1 parent 9e0c9ab
commit f6f8e65
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/browser/brave_screenshots/strategies/fullpage_strategy.cc b/browser/brave_screenshots/strategies/fullpage_strategy.cc
@@ -83,7 +83,7 @@ void FullPageStrategy::OnLayoutMetricsReceived(int width, int height) {
 }
 
 bool FullPageStrategy::DidClipScreenshot() const {
-  return screenshot_was_clipped_.value_or(false);
+  return screenshot_was_clipped_;
 }
 
 void FullPageStrategy::RequestFullPageScreenshot(int width, int height) {

diff --git a/browser/brave_screenshots/strategies/fullpage_strategy.h b/browser/brave_screenshots/strategies/fullpage_strategy.h
@@ -50,7 +50,7 @@ class FullPageStrategy : public BraveScreenshotStrategy,
   image_editor::ScreenshotCaptureCallback callback_;
   base::WeakPtr<content::WebContents> web_contents_;
   scoped_refptr<content::DevToolsAgentHost> devtools_host_;
-  std::optional<bool> screenshot_was_clipped_;
+  bool screenshot_was_clipped_ = false;
   int next_id_ = 1;
 };