DevTools: Introduce Page.getManifest remote debugging protocol method.

content/renderer/manifest/manifest_debug_info.h

Index: content/renderer/manifest/manifest_debug_info.h
diff --git a/content/renderer/manifest/manifest_debug_info.h b/content/renderer/manifest/manifest_debug_info.h
new file mode 100644
index 0000000000000000000000000000000000000000..c82eef9e160b66b1108aa3e0e43eec6a78405142
--- /dev/null
+++ b/content/renderer/manifest/manifest_debug_info.h
@@ -0,0 +1,32 @@
+// 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 CONTENT_RENDERER_MANIFEST_MANIFEST_DEBUG_INFO_H_
+#define CONTENT_RENDERER_MANIFEST_MANIFEST_DEBUG_INFO_H_
+
+#include <string>
+#include <vector>
+
+namespace content {
+
+struct ManifestDebugInfo {

[mlamouri (slow - plz ping), 2016/04/29 11:14:12]

style: can you add a comment explaining the purpose of the class.

[pfeldman, 2016/04/29 18:47:53]

Done.

+  struct Error {
+    Error(const std::string& message, int line, int column)

[mlamouri (slow - plz ping), 2016/04/29 11:14:12]

qq: could you take a && and do the same for Manifest::AddError(). Seems like we
could avoid one string copy. No need to do that in this CL though. Just a
thought.

[pfeldman, 2016/04/29 18:47:53]

Ack.

+        : message(message),
+          line(line),
+          column(column) {}
+    std::string message;
+    int line;
+    int column;
+  };
+
+  ManifestDebugInfo();
+  ~ManifestDebugInfo();
+  std::vector<Error> errors;
+  std::string raw_data;
+};
+
+} // namespace content
+
+#endif  // CONTENT_RENDERER_MANIFEST_MANIFEST_DEBUG_INFO_H_