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

Unified Diff: content/shell/browser/layout_test/layout_test_message_filter.cc

Issue 2403893002: Add layout test for reply property of notificationclick event (Closed)
Patch Set: Make reply field nullable in .idl files; add test cases for empty and null Created 4 years, 2 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: content/shell/browser/layout_test/layout_test_message_filter.cc
diff --git a/content/shell/browser/layout_test/layout_test_message_filter.cc b/content/shell/browser/layout_test/layout_test_message_filter.cc
index 630417c49b6b0480da84eb3b1be1695dacf18793..248f3e172659bb4f4d1051dd4b0734a37467ca45 100644
--- a/content/shell/browser/layout_test/layout_test_message_filter.cc
+++ b/content/shell/browser/layout_test/layout_test_message_filter.cc
@@ -125,11 +125,13 @@ void LayoutTestMessageFilter::OnSetDatabaseQuota(int quota) {
}
void LayoutTestMessageFilter::OnSimulateWebNotificationClick(
- const std::string& title, int action_index) {
+ const std::string& title,
+ int action_index,
+ const base::NullableString16& reply) {
LayoutTestNotificationManager* manager =
LayoutTestContentBrowserClient::Get()->GetLayoutTestNotificationManager();
if (manager)
- manager->SimulateClick(title, action_index);
+ manager->SimulateClick(title, action_index, reply);
}
void LayoutTestMessageFilter::OnSimulateWebNotificationClose(

Powered by Google App Engine
This is Rietveld 408576698