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

Unified Diff: chrome/browser/ui/browser_window.h

Issue 2400503002: [Translate] Integrate TranslateEventProto UMA logging into TranslateManager. (Closed)
Patch Set: fix trybots Created 4 years, 1 month 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
« no previous file with comments | « chrome/browser/ui/browser_commands.cc ('k') | chrome/browser/ui/cocoa/browser_window_cocoa.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_window.h
diff --git a/chrome/browser/ui/browser_window.h b/chrome/browser/ui/browser_window.h
index 46d3a952c89777ae735a24d4a5145084d8a7745d..8afc26f388afeef789736b9374c5466d1ea647dc 100644
--- a/chrome/browser/ui/browser_window.h
+++ b/chrome/browser/ui/browser_window.h
@@ -68,6 +68,18 @@ class WebContentsModalDialogHost;
enum class ImeWarningBubblePermissionStatus;
+enum class ShowTranslateBubbleResult {
+ // The translate bubble was successfully shown.
+ SUCCESS,
+
+ // The various reasons for which the translate bubble could fail to be shown.
+ BROWSER_WINDOW_NOT_VALID,
+ BROWSER_WINDOW_MINIMIZED,
+ BROWSER_WINDOW_NOT_ACTIVE,
+ WEB_CONTENTS_NOT_ACTIVE,
+ EDITABLE_FIELD_IS_ACTIVE,
+};
+
////////////////////////////////////////////////////////////////////////////////
// BrowserWindow interface
// An interface implemented by the "view" of the Browser window.
@@ -242,7 +254,7 @@ class BrowserWindow : public ui::BaseWindow {
//
// |is_user_gesture| is true when the bubble is shown on the user's deliberate
// action.
- virtual void ShowTranslateBubble(
+ virtual ShowTranslateBubbleResult ShowTranslateBubble(
content::WebContents* contents,
translate::TranslateStep step,
translate::TranslateErrors::Type error_type,
« no previous file with comments | « chrome/browser/ui/browser_commands.cc ('k') | chrome/browser/ui/cocoa/browser_window_cocoa.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698