chromium/src
0
Fork 0
Code Activity

Remove a NOTREACHED() in PDFiumEngine::SelectFindResult().

Browse Source 
It is expected that browser can still send this request even when there
are no find results.

BUG=498005

Change-Id: I234dc66739352ac35c2dfe072ac1f9cdee17bc2b
Reviewed-on: https://chromium-review.googlesource.com/c/1374690
Reviewed-by: Paul Meyer <paulmeyer@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617623}
This commit is contained in:
Lei Zhang
2018-12-18 21:21:04 +00:00
committed by Commit Bot
parent 9ffcb1890f
commit 65bc813377
1 changed files with 1 additions and 3 deletions

4
pdf/pdfium/pdfium_engine.cc

@ -2084,10 +2084,8 @@ void PDFiumEngine::AddFindResult(const PDFiumRange& result) {
} }
bool PDFiumEngine::SelectFindResult(bool forward) { bool PDFiumEngine::SelectFindResult(bool forward) {
if (find_results_.empty()) { if (find_results_.empty())
NOTREACHED();
return false; return false;
}
SelectionChangeInvalidator selection_invalidator(this); SelectionChangeInvalidator selection_invalidator(this);