From bf14e5bbc8a72aa59a0f4edb4043663096bb2e5b Mon Sep 17 00:00:00 2001 From: Max Karolinskiy Date: Wed, 21 Jan 2026 11:33:11 -0500 Subject: [PATCH] [cr145] Removes logging of CopyFromSurfaceError. Per review feedback. Chromium change: https://source.chromium.org/chromium/chromium/src/+/9d2b3244101ccd0eb8321d3027c648ec4a1b14e2 commit 9d2b3244101ccd0eb8321d3027c648ec4a1b14e2 Author: Dave Tapuska Date: Thu Jan 15 07:32:24 2026 -0800 Set timeout from multi source page context fetcher - Change the result code on CopyFromSurface so we can log a uma histogram instead of an arbitrary string - Pass timeout to viz, set the browser side timeout to be 500ms later to give the IPC enough time. Bug: b:447589565 --- components/ai_chat/content/browser/page_content_fetcher.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/components/ai_chat/content/browser/page_content_fetcher.cc b/components/ai_chat/content/browser/page_content_fetcher.cc index d7f0e35d2ba..27212c78162 100644 --- a/components/ai_chat/content/browser/page_content_fetcher.cc +++ b/components/ai_chat/content/browser/page_content_fetcher.cc @@ -469,8 +469,6 @@ class PageContentFetcherInternal { void OnScreenshot(FetchPageContentCallback callback, const content::CopyFromSurfaceResult& result) { if (!result.has_value()) { - LOG(ERROR) << "RenderWidgetHostView::CopyFromSurface failed with: " - << result.error(); std::move(callback).Run("", false, ""); return; }