Commit 718ab74b authored by Oksana Zhuravlova's avatar Oksana Zhuravlova Committed by Commit Bot

[mojo][bindings] Remove redundant deserialization error description

Since ReportValidationError() already logs the description from its
|context| parameter, there is no need to pass the description separately.

Test: temporarily triggered the deserialization error and confirmed the error message has only one instance of interface name and method number

Change-Id: I813052bbaf4485d1fd269174b3a645535f548c7c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2157721
Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
Reviewed-by: default avatarKen Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#761122}
parent e8dff94a
...@@ -104,7 +104,7 @@ void ReportValidationErrorForMessage(mojo::Message* message, ...@@ -104,7 +104,7 @@ void ReportValidationErrorForMessage(mojo::Message* message,
is_response ? " response" : ""); is_response ? " response" : "");
ValidationContext validation_context(nullptr, 0, 0, 0, message, ValidationContext validation_context(nullptr, 0, 0, 0, message,
description.c_str()); description.c_str());
ReportValidationError(&validation_context, error, description.c_str()); ReportValidationError(&validation_context, error);
} }
ScopedSuppressValidationErrorLoggingForTests ScopedSuppressValidationErrorLoggingForTests
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment