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

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

Issue 2679533002: Added unit test for WebShareTargetPickerView. (Closed)
Patch Set: Created 3 years, 10 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..c482024080ddb62ee1ef7460bf1fc97db4b02c7e 100644
--- a/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
+++ b/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
@@ -15,6 +15,14 @@
#include "ui/base/ui_base_types.h"
#include "ui/views/window/dialog_delegate.h"
+namespace views {
+class TableView;
+}
+
+namespace {
+class WebShareTargetPickerViewTest;
+}
+
// Dialog that presents the user with a list of share target apps. Allows the
// user to pick one target to be opened and have data passed to it.
//
@@ -50,6 +58,11 @@ class WebShareTargetPickerView : public views::DialogDelegateView,
void SetObserver(ui::TableModelObserver* observer) override;
private:
+ // For access to |table_|.
+ friend class WebShareTargetPickerViewTest;
+
+ views::TableView* table_;
+
const std::vector<base::string16> targets_;
base::Callback<void(SharePickerResult)> close_callback_;

Powered by Google App Engine
This is Rietveld 408576698