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

Unified Diff: mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom

Issue 2312813002: Limit Mojo messages recursion depth (Closed)
Patch Set: Update comment to match arg name change Created 4 years, 3 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: mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom
diff --git a/mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom b/mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom
index 705650bc18774a4cbc672337735c489d2e58c9b3..5d8296a7b10993052c526fb8df1ad1459ee26b8d 100644
--- a/mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom
+++ b/mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom
@@ -92,6 +92,10 @@ interface ConformanceTestInterface {
Method18(UnionA? param0);
};
+interface RecursionDepthTestInterface {
yzshen1 2016/09/06 17:03:10 I think it may make sense to add it to Conformance
tibell 2016/09/07 05:41:20 Moved it into ConformanceTestInterface. I agree w
+ Method0(Recursive recursive);
+};
+
struct BasicStruct {
int32 a;
};
@@ -117,3 +121,8 @@ struct StructWithEnum {
A, B, C, D
};
};
+
+// This is used to test that deeply recursive structures don't blow the stack.
+struct Recursive {
+ Recursive? recursive;
+};

Powered by Google App Engine
This is Rietveld 408576698