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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java

Issue 2866273003: [Android] Improve sad tab strings (Closed)
Patch Set: Fix spacing Created 3 years, 7 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/android/java/src/org/chromium/chrome/browser/tab/Tab.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java b/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
index ca98511f3688a325ce42876534608273994dc361..d7f93eb551645a082d3f21cbfb8a4cce53b6ded5 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
@@ -1695,9 +1695,16 @@ public class Tab
*/
protected void showSadTab() {
if (getContentViewCore() != null) {
+ // If the tab has crashed twice in a row change the sad tab view to the "Send Feedback"
+ // version and change the onClickListener.
+ final boolean showSendFeedbackView = mSadTabSuccessiveRefreshCounter >= 1;
+
OnClickListener suggestionAction = new OnClickListener() {
@Override
public void onClick(View view) {
+ SadTabViewFactory.recordEvent(showSendFeedbackView,
+ SadTabViewFactory.SAD_TAB_EVENT_HELP_LINK_CLICKED);
+
Activity activity = mWindowAndroid.getActivity().get();
assert activity != null;
HelpAndFeedback.getInstance(activity).show(activity,
@@ -1706,14 +1713,12 @@ public class Tab
}
};
- // If the tab has crashed twice in a row change the button to "Send Feedback" and
- // change the onClickListener.
- final boolean showSendFeedbackButton = mSadTabSuccessiveRefreshCounter >= 1;
OnClickListener buttonAction = new OnClickListener() {
-
@Override
public void onClick(View v) {
- if (showSendFeedbackButton) {
+ SadTabViewFactory.recordEvent(
+ showSendFeedbackView, SadTabViewFactory.SAD_TAB_EVENT_BUTTON_CLICKED);
+ if (showSendFeedbackView) {
getActivity().startHelpAndFeedback(Tab.this, "MobileSadTabFeedback");
} else {
reload();
@@ -1725,14 +1730,16 @@ public class Tab
assert mSadTabView == null;
mSadTabView = SadTabViewFactory.createSadTabView(mThemedApplicationContext,
- suggestionAction, buttonAction, showSendFeedbackButton
- ? R.string.sad_tab_send_feedback_label : R.string.sad_tab_reload_label);
+ suggestionAction, buttonAction, showSendFeedbackView);
mSadTabSuccessiveRefreshCounter++;
// Show the sad tab inside ContentView.
getContentViewCore().getContainerView().addView(
mSadTabView, new FrameLayout.LayoutParams(
LayoutParams.MATCH_PARENT, LayoutParams.MATCH_PARENT));
notifyContentChanged();
+
+ SadTabViewFactory.recordEvent(
+ showSendFeedbackView, SadTabViewFactory.SAD_TAB_EVENT_DISPLAYED);
}
FullscreenManager fullscreenManager = getFullscreenManager();
if (fullscreenManager != null) {

Powered by Google App Engine
This is Rietveld 408576698