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

Unified Diff: mojo/public/c/bindings/lib/message.c

Issue 2232833003: Change the canonical way to include the C bindings headers to <mojo/bindings/*.h>. (Closed) Base URL: https://github.com/domokit/mojo.git@work791_mojo_tests
Patch Set: rebased Created 4 years, 4 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 | « mojo/public/c/bindings/lib/map.c ('k') | mojo/public/c/bindings/lib/struct.c » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/c/bindings/lib/message.c
diff --git a/mojo/public/c/bindings/lib/message.c b/mojo/public/c/bindings/lib/message.c
deleted file mode 100644
index 6c766e3009fe7b192483c4d6a116e3bfcd0cb8b3..0000000000000000000000000000000000000000
--- a/mojo/public/c/bindings/lib/message.c
+++ /dev/null
@@ -1,74 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "mojo/public/c/bindings/message.h"
-
-#include <stdint.h>
-
-#include "mojo/public/c/bindings/struct.h"
-
-MojomValidationResult MojomMessage_ValidateHeader(const void* in_buf,
- uint32_t in_buf_size) {
- const struct MojomStructHeader* header =
- (const struct MojomStructHeader*)in_buf;
-
- if (in_buf_size < sizeof(struct MojomStructHeader) ||
- in_buf_size < header->num_bytes)
- return MOJOM_VALIDATION_ILLEGAL_MEMORY_RANGE;
-
- const struct MojomMessage* msg = (const struct MojomMessage*)in_buf;
- if (header->version == 0u) {
- if (header->num_bytes != sizeof(struct MojomMessage)) {
- return MOJOM_VALIDATION_UNEXPECTED_STRUCT_HEADER;
- }
-
- // Version 0 has no request id and should not have either of these flags.
- if ((msg->flags & MOJOM_MESSAGE_FLAGS_EXPECTS_RESPONSE) ||
- (msg->flags & MOJOM_MESSAGE_FLAGS_IS_RESPONSE)) {
- return MOJOM_VALIDATION_MESSAGE_HEADER_MISSING_REQUEST_ID;
- }
- } else if (header->version == 1u) {
- if (header->num_bytes != sizeof(struct MojomMessageWithRequestId)) {
- return MOJOM_VALIDATION_UNEXPECTED_STRUCT_HEADER;
- }
- } else if (header->version > 1u) {
- if (header->num_bytes < sizeof(struct MojomMessageWithRequestId)) {
- return MOJOM_VALIDATION_UNEXPECTED_STRUCT_HEADER;
- }
- }
-
- // Mutually exclusive flags.
- if ((msg->flags & MOJOM_MESSAGE_FLAGS_EXPECTS_RESPONSE) &&
- (msg->flags & MOJOM_MESSAGE_FLAGS_IS_RESPONSE)) {
- return MOJOM_VALIDATION_MESSAGE_HEADER_INVALID_FLAGS;
- }
-
- // Accept unknown versions of the message header to be future-proof.
- return MOJOM_VALIDATION_ERROR_NONE;
-}
-
-MojomValidationResult MojomMessage_ValidateRequestExpectingResponse(
- const void* in_buf) {
- const struct MojomMessage* msg = (const struct MojomMessage*)in_buf;
- return (msg->flags & MOJOM_MESSAGE_FLAGS_EXPECTS_RESPONSE)
- ? MOJOM_VALIDATION_ERROR_NONE
- : MOJOM_VALIDATION_MESSAGE_HEADER_INVALID_FLAGS;
-}
-
-MojomValidationResult MojomMessage_ValidateRequestWithoutResponse(
- const void* in_buf) {
- const struct MojomMessage* msg = (const struct MojomMessage*)in_buf;
- return (!(msg->flags & MOJOM_MESSAGE_FLAGS_EXPECTS_RESPONSE) &&
- !(msg->flags & MOJOM_MESSAGE_FLAGS_IS_RESPONSE))
- ? MOJOM_VALIDATION_ERROR_NONE
- : MOJOM_VALIDATION_MESSAGE_HEADER_INVALID_FLAGS;
-}
-
-MojomValidationResult MojomMessage_ValidateResponse(
- const void* in_buf) {
- const struct MojomMessage* msg = (const struct MojomMessage*)in_buf;
- return (msg->flags & MOJOM_MESSAGE_FLAGS_IS_RESPONSE)
- ? MOJOM_VALIDATION_ERROR_NONE
- : MOJOM_VALIDATION_MESSAGE_HEADER_INVALID_FLAGS;
-}
« no previous file with comments | « mojo/public/c/bindings/lib/map.c ('k') | mojo/public/c/bindings/lib/struct.c » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698