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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/input/MultiFieldTimePickerDialog.java

Issue 260903006: Dismissing of date/time dialogs should not set the value (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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: content/public/android/java/src/org/chromium/content/browser/input/MultiFieldTimePickerDialog.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/input/MultiFieldTimePickerDialog.java b/content/public/android/java/src/org/chromium/content/browser/input/MultiFieldTimePickerDialog.java
index 77904d3548d45b6991e3f5140b04d2e8161dbdfc..e415a6219a352ff78bee75b3af7ebc0786c3760c 100644
--- a/content/public/android/java/src/org/chromium/content/browser/input/MultiFieldTimePickerDialog.java
+++ b/content/public/android/java/src/org/chromium/content/browser/input/MultiFieldTimePickerDialog.java
@@ -8,7 +8,6 @@ import android.app.AlertDialog;
import android.content.Context;
import android.content.DialogInterface;
import android.content.DialogInterface.OnClickListener;
-import android.os.Build;
import android.view.LayoutInflater;
import android.view.View;
import android.widget.NumberPicker;
@@ -261,18 +260,6 @@ public class MultiFieldTimePickerDialog
mListener.onTimeSet(hour, minute, sec, milli);
}
- @Override
- protected void onStop() {
- if (Build.VERSION.SDK_INT >= 16) {
- // The default behavior of dialogs changed in JellyBean and onwards.
- // Dismissing a dialog (by pressing back for example)
- // applies the chosen date. This code is added here so that the custom
- // pickers behave the same as the internal DatePickerDialog.
- notifyDateSet();
- }
- super.onStop();
- }
-
private static class NumberFormatter implements NumberPicker.Formatter {
private final String mFormat;

Powered by Google App Engine
This is Rietveld 408576698