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

Unified Diff: chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h

Issue 2617633004: Fix AccessibilityExtensionLoader's handling of locked screen (Closed)
Patch Set: Test removals. 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/guest_view/web_view/chrome_web_view_guest_delegate.h
diff --git a/chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h b/chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h
index 682f4a233738427bd9b9d1bf3d5e01fadeea5d4b..f64e07c215725182569f214a9f4ceb306646e6f2 100644
--- a/chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h
+++ b/chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h
@@ -11,10 +11,6 @@
#include "extensions/browser/guest_view/web_view/web_view_guest.h"
#include "extensions/browser/guest_view/web_view/web_view_guest_delegate.h"
-#if defined(OS_CHROMEOS)
-#include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
-#endif
-
class RenderViewContextMenuBase;
namespace ui {
@@ -30,7 +26,6 @@ class ChromeWebViewGuestDelegate : public WebViewGuestDelegate {
// WebViewGuestDelegate implementation.
bool HandleContextMenu(const content::ContextMenuParams& params) override;
- void OnDidInitialize() override;
void OnShowContextMenu(int request_id) override;
bool ShouldHandleFindRequestsForEmbedder() const override;
@@ -47,14 +42,6 @@ class ChromeWebViewGuestDelegate : public WebViewGuestDelegate {
static std::unique_ptr<base::ListValue> MenuModelToValue(
const ui::SimpleMenuModel& menu_model);
- void InjectChromeVoxIfNeeded(content::RenderViewHost* render_view_host);
-
-#if defined(OS_CHROMEOS)
- // Notification of a change in the state of an accessibility setting.
- void OnAccessibilityStatusChanged(
- const chromeos::AccessibilityStatusEventDetails& details);
-#endif
-
// A counter to generate a unique request id for a context menu request.
// We only need the ids to be unique for a given WebViewGuest.
int pending_context_menu_request_id_;
@@ -63,15 +50,6 @@ class ChromeWebViewGuestDelegate : public WebViewGuestDelegate {
// shown. This is .reset() after ShowContextMenu().
std::unique_ptr<RenderViewContextMenuBase> pending_menu_;
-#if defined(OS_CHROMEOS)
- // Set to |true| if ChromeVox was already injected in main frame.
- bool chromevox_injected_ = false;
-
- // Subscription to receive notifications on changes to a11y settings.
- std::unique_ptr<chromeos::AccessibilityStatusSubscription>
- accessibility_subscription_;
-#endif
-
WebViewGuest* const web_view_guest_;
std::unique_ptr<gfx::Point> context_menu_position_;

Powered by Google App Engine
This is Rietveld 408576698