Commit 55015efd authored by Ankit Kumar's avatar Ankit Kumar Committed by Commit Bot

Remove links from tab order in Print Preview

Bug: 1084957
Change-Id: Ida67162676737644e987a43aaff5cf713a75301e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212221Reviewed-by: default avatarLei Zhang <thestig@chromium.org>
Commit-Queue: Ankit Kumar 🌪️ <ankk@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#771318}
parent fbed9529
...@@ -2707,7 +2707,8 @@ void PDFiumEngine::LoadForm() { ...@@ -2707,7 +2707,8 @@ void PDFiumEngine::LoadForm() {
kFormHighlightColor); kFormHighlightColor);
FPDF_SetFormFieldHighlightAlpha(form(), kFormHighlightAlpha); FPDF_SetFormFieldHighlightAlpha(form(), kFormHighlightAlpha);
if (base::FeatureList::IsEnabled(features::kTabAcrossPDFAnnotations)) { if (base::FeatureList::IsEnabled(features::kTabAcrossPDFAnnotations) &&
!client_->IsPrintPreview()) {
static constexpr FPDF_ANNOTATION_SUBTYPE kFocusableAnnotSubtypes[] = { static constexpr FPDF_ANNOTATION_SUBTYPE kFocusableAnnotSubtypes[] = {
FPDF_ANNOT_LINK, FPDF_ANNOT_HIGHLIGHT, FPDF_ANNOT_WIDGET}; FPDF_ANNOT_LINK, FPDF_ANNOT_HIGHLIGHT, FPDF_ANNOT_WIDGET};
FPDF_BOOL ret = FPDFAnnot_SetFocusableSubtypes( FPDF_BOOL ret = FPDFAnnot_SetFocusableSubtypes(
......
...@@ -146,8 +146,7 @@ void PreviewModeClient::FormTextFieldFocusChange(bool in_focus) { ...@@ -146,8 +146,7 @@ void PreviewModeClient::FormTextFieldFocusChange(bool in_focus) {
} }
bool PreviewModeClient::IsPrintPreview() { bool PreviewModeClient::IsPrintPreview() {
NOTREACHED(); return true;
return false;
} }
float PreviewModeClient::GetToolbarHeightInScreenCoords() { float PreviewModeClient::GetToolbarHeightInScreenCoords() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment