headless: Add error reporter utility

headless/public/util/error_reporter.h

+// 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.
+
+#ifndef HEADLESS_PUBLIC_UTIL_ERROR_REPORTER_H_
+#define HEADLESS_PUBLIC_UTIL_ERROR_REPORTER_H_
+
+#include <string>
+#include <vector>
+
+#include "headless/public/headless_export.h"
+
+namespace headless {
+
+// Tracks errors which are encountered while parsing client API types.
+class HEADLESS_EXPORT ErrorReporter {
+ public:
+  ErrorReporter();
+  ~ErrorReporter();
+
+  // Enter a new nested parsing context.
+  void Push();

[altimin, 2016/04/11 14:49:57]

Can we merge Push and SetName?

[Sami, 2016/04/11 15:04:10]

When parsing a large object the expected usage pattern is like this:

errors.Push();
errors.SetName("field1");
if (failed) errors.AddError("parsing failed");
errors.SetName("field2");
if (failed) errors.AddError("parsing failed");
errors.SetName("field3");
if (failed) errors.AddError("parsing failed");
...
errors.Pop();

Merging Push() and SetName() would add some redundant stack operations to each
field, which I'd like to avoid.

+
+  // Leave the current parsing context, returning to the previous one.
+  void Pop();
+
+  // Set the name of the current parsing context. |name| must be a string with
+  // application lifetime.
+  void SetName(const char* name);
+
+  // Report an error in the current parsing context.
+  void AddError(const char* description);
+
+  // Returns true if any errors have been reported so far.
+  bool HasErrors() const;
+
+  // Returns a list of reported errors.
+  const std::vector<std::string>& errors() const { return errors_; }
+
+ private:
+  std::vector<const char*> path_;
+  std::vector<std::string> errors_;
+};
+
+}  // namespace headless
+
+#endif  // HEADLESS_PUBLIC_UTIL_ERROR_REPORTER_H_