Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5066)

Unified Diff: chrome/browser/ui/views/location_bar/open_pdf_in_reader_view.cc

Issue 2672463003: Remove dead code since PDFWebContentsHelper::ShowOpenInReaderPrompt isn't called anymore. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/location_bar/open_pdf_in_reader_view.cc
diff --git a/chrome/browser/ui/views/location_bar/open_pdf_in_reader_view.cc b/chrome/browser/ui/views/location_bar/open_pdf_in_reader_view.cc
deleted file mode 100644
index 58f9375c661040fa736dd6c80293d7c8c68c1118..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/location_bar/open_pdf_in_reader_view.cc
+++ /dev/null
@@ -1,95 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/ui/views/location_bar/open_pdf_in_reader_view.h"
-
-#include "chrome/browser/ui/views/open_pdf_in_reader_bubble_view.h"
-#include "chrome/grit/generated_resources.h"
-#include "components/pdf/browser/open_pdf_in_reader_prompt_client.h"
-#include "components/pdf/browser/pdf_web_contents_helper.h"
-#include "ui/accessibility/ax_node_data.h"
-#include "ui/base/l10n/l10n_util.h"
-#include "ui/gfx/color_utils.h"
-#include "ui/gfx/paint_vector_icon.h"
-#include "ui/gfx/vector_icons_public.h"
-#include "ui/native_theme/native_theme.h"
-#include "ui/views/widget/widget.h"
-
-OpenPDFInReaderView::OpenPDFInReaderView() : bubble_(nullptr), model_(nullptr) {
- SetFocusBehavior(FocusBehavior::ACCESSIBLE_ONLY);
- SetTooltipText(l10n_util::GetStringUTF16(IDS_PDF_BUBBLE_OPEN_IN_READER_LINK));
-}
-
-OpenPDFInReaderView::~OpenPDFInReaderView() {
- if (bubble_)
- bubble_->GetWidget()->RemoveObserver(this);
-}
-
-void OpenPDFInReaderView::Update(content::WebContents* web_contents) {
- model_ = nullptr;
- if (web_contents) {
- pdf::PDFWebContentsHelper* pdf_tab_helper =
- pdf::PDFWebContentsHelper::FromWebContents(web_contents);
- model_ = pdf_tab_helper->open_in_reader_prompt();
- }
-
- SetVisible(!!model_);
-
- // Hide the bubble if it is currently shown and the icon is hidden.
- if (!model_ && bubble_)
- bubble_->GetWidget()->Hide();
-}
-
-void OpenPDFInReaderView::ShowBubble() {
- if (bubble_)
- return;
-
- DCHECK(model_);
- bubble_ = new OpenPDFInReaderBubbleView(this, model_);
- views::BubbleDialogDelegateView::CreateBubble(bubble_);
- bubble_->GetWidget()->AddObserver(this);
- bubble_->GetWidget()->Show();
-}
-
-void OpenPDFInReaderView::GetAccessibleNodeData(ui::AXNodeData* node_data) {
- ImageView::GetAccessibleNodeData(node_data);
- node_data->SetName(l10n_util::GetStringUTF8(IDS_ACCNAME_OPEN_PDF_IN_READER));
- node_data->role = ui::AX_ROLE_BUTTON;
-}
-
-bool OpenPDFInReaderView::OnMousePressed(const ui::MouseEvent& event) {
- // Show the bubble on mouse release; that is standard button behavior.
- return true;
-}
-
-void OpenPDFInReaderView::OnMouseReleased(const ui::MouseEvent& event) {
- if (event.IsOnlyLeftMouseButton() && HitTestPoint(event.location()))
- ShowBubble();
-}
-
-bool OpenPDFInReaderView::OnKeyPressed(const ui::KeyEvent& event) {
- if (event.key_code() != ui::VKEY_SPACE &&
- event.key_code() != ui::VKEY_RETURN) {
- return false;
- }
-
- ShowBubble();
- return true;
-}
-
-void OpenPDFInReaderView::OnNativeThemeChanged(
- const ui::NativeTheme* native_theme) {
- SetImage(gfx::CreateVectorIcon(
- gfx::VectorIconId::PDF,
- color_utils::DeriveDefaultIconColor(native_theme->GetSystemColor(
- ui::NativeTheme::kColorId_TextfieldDefaultColor))));
-}
-
-void OpenPDFInReaderView::OnWidgetDestroying(views::Widget* widget) {
- if (!bubble_)
- return;
-
- bubble_->GetWidget()->RemoveObserver(this);
- bubble_ = nullptr;
-}

Powered by Google App Engine
This is Rietveld 408576698