diff --git a/browser/ai_chat/upload_file_helper.cc b/browser/ai_chat/upload_file_helper.cc index c112f8552c1..e95002aa7ec 100644 --- a/browser/ai_chat/upload_file_helper.cc +++ b/browser/ai_chat/upload_file_helper.cc @@ -250,11 +250,8 @@ void UploadFileHelper::MultiFilesSelected( base::OnceCallback>, base::FilePath, std::optional>)> callback, - std::tuple>, base::FilePath> - result) { - auto file_data = std::get<0>(result); - auto filepath = std::get<1>(result); - + std::optional> file_data, + base::FilePath filepath) { if (!file_data) { std::move(callback).Run(std::make_tuple( std::nullopt, std::move(filepath), std::nullopt)); @@ -311,43 +308,41 @@ void UploadFileHelper::FileSelectionCanceled() { } } -void UploadFileHelper::OnFileRead( - std::tuple>, base::FilePath> result) { +void UploadFileHelper::OnFileRead(std::optional> data, + base::FilePath path) { DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); - auto file_data = std::get<0>(result); - if (!file_data) { + if (!data) { std::move(upload_file_callback_).Run(std::nullopt); return; } // Determine file type based on extension and validate PDF content - auto file_type_opt = DetermineFileType(std::get<1>(result), *file_data); + auto file_type_opt = DetermineFileType(path, *data); if (file_type_opt && (*file_type_opt == mojom::UploadedFileType::kPdf || *file_type_opt == mojom::UploadedFileType::kText)) { // Return raw data; text extraction happens via ProcessPdfFile or // ProcessTextFile mojo endpoint. std::vector files; - files.push_back(mojom::UploadedFile::New(std::get<1>(result).AsUTF8Unsafe(), - file_data->size(), *file_data, - *file_type_opt, std::nullopt)); + files.push_back(mojom::UploadedFile::New(path.AsUTF8Unsafe(), data->size(), + *data, *file_type_opt, + std::nullopt)); std::move(upload_file_callback_).Run(std::make_optional(std::move(files))); } else if (file_type_opt && *file_type_opt == mojom::UploadedFileType::kImage) { // For images, process them as before UploadFileHelper::ProcessImageData( - &data_decoder_, *file_data, + &data_decoder_, *data, base::BindOnce(&UploadFileHelper::OnImageEncoded, - weak_ptr_factory_.GetWeakPtr(), - std::get<1>(result).AsUTF8Unsafe())); + weak_ptr_factory_.GetWeakPtr(), path.AsUTF8Unsafe())); } else { // Include as empty stub so the frontend can detect the unsupported // file and show an error (as opposed to nullopt which means the user // cancelled the file picker). std::vector files; - files.push_back(mojom::UploadedFile::New( - std::get<1>(result).AsUTF8Unsafe(), 0, std::vector(), - mojom::UploadedFileType::kText, std::nullopt)); + files.push_back( + mojom::UploadedFile::New(path.AsUTF8Unsafe(), 0, std::vector(), + mojom::UploadedFileType::kText, std::nullopt)); std::move(upload_file_callback_).Run(std::make_optional(std::move(files))); } } diff --git a/browser/ai_chat/upload_file_helper.h b/browser/ai_chat/upload_file_helper.h index b4d8d688582..2120bfaaef1 100644 --- a/browser/ai_chat/upload_file_helper.h +++ b/browser/ai_chat/upload_file_helper.h @@ -68,8 +68,8 @@ class UploadFileHelper : public ui::SelectFileDialog::Listener { const std::vector& files) override; void FileSelectionCanceled() override; - void OnFileRead( - std::tuple>, base::FilePath> result); + void OnFileRead(std::optional> data, + base::FilePath path); void OnImageEncoded(std::string filename, std::optional> output); diff --git a/components/brave_news/browser/feed_fetcher.cc b/components/brave_news/browser/feed_fetcher.cc index 9dfb1eeeaf2..cdf73a0633f 100644 --- a/components/brave_news/browser/feed_fetcher.cc +++ b/components/brave_news/browser/feed_fetcher.cc @@ -225,16 +225,16 @@ void FeedFetcher::OnFetchFeedFetchedFeed( void FeedFetcher::OnFetchFeedFetchedAll(FetchFeedCallback callback, std::vector results) { base::ThreadPool::PostTaskAndReplyWithResult( - FROM_HERE, base::BindOnce(&CombineFeedSourceResults, std::move(results)), + FROM_HERE, {}, + base::BindOnce(&CombineFeedSourceResults, std::move(results)), base::BindOnce( [](base::WeakPtr fetcher, FetchFeedCallback callback, - std::tuple result) { + FeedItems items, ETags tags) { // If we've been destroyed, don't run the callback. if (!fetcher) { return; } - std::move(callback).Run(std::move(std::get<0>(result)), - std::move(std::get<1>(result))); + std::move(callback).Run(std::move(items), std::move(tags)); }, weak_ptr_factory_.GetWeakPtr(), std::move(callback))); }