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

Unified Diff: chrome/browser/ui/views/webshare/webshare_target_picker_view.h

Issue 2667803002: Picker takes a vector of pairs, and passes back the user chosen target. (Closed)
Patch Set: Move replacing of placeholders to after picker 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/ui/views/webshare/webshare_target_picker_view.h
diff --git a/chrome/browser/ui/views/webshare/webshare_target_picker_view.h b/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
index 324955ebf5d36df8916559983b0abb052a5de22e..a4dcc7fde19de4d86ec316946f7e59cbd14cb829 100644
--- a/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
+++ b/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
@@ -13,6 +13,7 @@
#include "chrome/browser/ui/browser_dialogs.h"
#include "ui/base/models/table_model.h"
#include "ui/base/ui_base_types.h"
+#include "ui/views/controls/table/table_view.h"
#include "ui/views/window/dialog_delegate.h"
// Dialog that presents the user with a list of share target apps. Allows the
@@ -27,9 +28,10 @@ class WebShareTargetPickerView : public views::DialogDelegateView,
// |targets| is a list of app titles that will be shown in a list. Calls
// |close_callback| with SHARE if an app was chosen, or CANCEL if the dialog
// was cancelled.
- WebShareTargetPickerView(
+ explicit WebShareTargetPickerView(
const std::vector<base::string16>& targets,
- const base::Callback<void(SharePickerResult)>& close_callback);
+ const base::Callback<void(base::Optional<base::string16>)>&
+ close_callback);
~WebShareTargetPickerView() override;
// views::View overrides:
@@ -50,9 +52,11 @@ class WebShareTargetPickerView : public views::DialogDelegateView,
void SetObserver(ui::TableModelObserver* observer) override;
private:
+ views::TableView* table_;
+
const std::vector<base::string16> targets_;
- base::Callback<void(SharePickerResult)> close_callback_;
+ base::Callback<void(base::Optional<base::string16>)> close_callback_;
DISALLOW_COPY_AND_ASSIGN(WebShareTargetPickerView);
};

Powered by Google App Engine
This is Rietveld 408576698