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

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

Issue 2679533002: Added unit test for WebShareTargetPickerView. (Closed)
Patch Set: Remove double-click on empty list logic + test. 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
« no previous file with comments | « no previous file | chrome/browser/ui/views/webshare/webshare_target_picker_view_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 bb455fe17136d9fa1aa29a5580647650208f5cd8..18464cf3759ff000f7ab2fbe7c0ef92490f669cd 100644
--- a/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
+++ b/chrome/browser/ui/views/webshare/webshare_target_picker_view.h
@@ -22,6 +22,10 @@ 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.
//
@@ -59,6 +63,9 @@ class WebShareTargetPickerView : public views::DialogDelegateView,
void OnDoubleClick() override;
private:
+ // For access to |table_|.
+ friend class WebShareTargetPickerViewTest;
+
views::TableView* table_ = nullptr;
const std::vector<std::pair<base::string16, GURL>> targets_;
« no previous file with comments | « no previous file | chrome/browser/ui/views/webshare/webshare_target_picker_view_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698