Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java b/chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java |
| index 4367e01e676ffdaa3842cc76284deeb9a91ffdee..4a9768868791c00bf1fc856578681a727534058d 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java |
| @@ -31,6 +31,7 @@ import org.chromium.chrome.browser.identity.UuidBasedUniqueIdentificationGenerat |
| import org.chromium.chrome.browser.invalidation.UniqueIdInvalidationClientNameGenerator; |
| import org.chromium.chrome.browser.multiwindow.MultiWindowUtils; |
| import org.chromium.chrome.browser.net.spdyproxy.DataReductionProxySettings; |
| +import org.chromium.chrome.browser.photo_picker.PhotoPickerDialog; |
| import org.chromium.chrome.browser.preferences.PrefServiceBridge; |
| import org.chromium.chrome.browser.rlz.RevenueStats; |
| import org.chromium.chrome.browser.services.AccountsChangedReceiver; |
| @@ -42,7 +43,6 @@ import org.chromium.printing.PrintDocumentAdapterWrapper; |
| import org.chromium.printing.PrintingControllerImpl; |
| import org.chromium.ui.PhotoPickerListener; |
| import org.chromium.ui.UiUtils; |
| -import org.chromium.ui.widget.Toast; |
| /** |
| * Handles the initialization dependences of the browser process. This is meant to handle the |
| @@ -60,6 +60,7 @@ public class ProcessInitializationHandler { |
| private boolean mInitializedPostNative; |
| private boolean mInitializedDeferredStartupTasks; |
| private DevToolsServer mDevToolsServer; |
| + private PhotoPickerDialog mDialog; |
|
Ted C
2017/04/06 23:52:15
will this ever need to be used outside of the dele
Finnur
2017/04/07 13:52:20
Done.
|
| /** |
| * @return The ProcessInitializationHandler for use during the lifetime of the browser process. |
| @@ -162,12 +163,15 @@ public class ProcessInitializationHandler { |
| @Override |
| public void showPhotoPicker( |
| Context context, PhotoPickerListener listener, boolean allowMultiple) { |
| - Toast toast = Toast.makeText(context, "Not implemented!", Toast.LENGTH_SHORT); |
| - toast.show(); |
| + mDialog = new PhotoPickerDialog(context, listener, allowMultiple); |
| + mDialog.show(); |
| } |
| @Override |
| - public void dismissPhotoPicker() {} |
| + public void dismissPhotoPicker() { |
| + mDialog.dismiss(); |
| + mDialog = null; |
| + } |
| }); |
| } |
| } |