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

Unified Diff: chrome/browser/ui/toolbar/media_router_action_unittest.cc

Issue 2176613003: [Media Router] Clean up issues related code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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/toolbar/media_router_action_unittest.cc
diff --git a/chrome/browser/ui/toolbar/media_router_action_unittest.cc b/chrome/browser/ui/toolbar/media_router_action_unittest.cc
index d09f0225681e175132a815f50659208f07e74950..9bd2e230a34d58260d2d72266cc1ae1d4f8e77c5 100644
--- a/chrome/browser/ui/toolbar/media_router_action_unittest.cc
+++ b/chrome/browser/ui/toolbar/media_router_action_unittest.cc
@@ -68,34 +68,15 @@ class MediaRouterActionUnitTest : public MediaRouterTest {
public:
MediaRouterActionUnitTest()
: toolbar_model_(nullptr),
- fake_issue_notification_(media_router::Issue(
- "title notification",
- "message notification",
- media_router::IssueAction(media_router::IssueAction::TYPE_DISMISS),
- std::vector<media_router::IssueAction>(),
- "route_id",
- media_router::Issue::NOTIFICATION,
- false,
- -1)),
- fake_issue_warning_(
- media_router::Issue("title warning",
- "message warning",
- media_router::IssueAction(
- media_router::IssueAction::TYPE_LEARN_MORE),
- std::vector<media_router::IssueAction>(),
- "route_id",
- media_router::Issue::WARNING,
- false,
- 12345)),
- fake_issue_fatal_(media_router::Issue(
- "title fatal",
- "message fatal",
- media_router::IssueAction(media_router::IssueAction::TYPE_DISMISS),
- std::vector<media_router::IssueAction>(),
- "route_id",
- media_router::Issue::FATAL,
- true,
- -1)),
+ fake_issue_notification_("title notification",
+ media_router::Issue::ActionType::DISMISS,
+ media_router::Issue::Severity::NOTIFICATION),
+ fake_issue_warning_("title warning",
+ media_router::Issue::ActionType::LEARN_MORE,
+ media_router::Issue::Severity::WARNING),
+ fake_issue_fatal_("title fatal",
+ media_router::Issue::ActionType::DISMISS,
+ media_router::Issue::Severity::FATAL),
fake_source1_("fakeSource1"),
fake_source2_("fakeSource2"),
active_icon_(

Powered by Google App Engine
This is Rietveld 408576698