Initial CL for talking to the WebAPK server to generate WebAPK

content/public/common/manifest_util.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 CONTENT_PUBLIC_COMMON_MANIFEST_UTIL_H_
+#define CONTENT_PUBLIC_COMMON_MANIFEST_UTIL_H_
+
+#include <string>
+
+#include "content/common/content_export.h"
+#include "third_party/WebKit/public/platform/WebDisplayMode.h"
+#include "third_party/WebKit/public/platform/modules/screen_orientation/WebScreenOrientationLockType.h"
+
+namespace content {
+

[no sievers, 2016/08/05 17:53:08]

Feels every so slightly odd that the content public api exposes these blink
enum<->string helper (and not for example Blink itself). But then again, the
types are exposed a lot in the content-public api here, so also not totally
unreasonable.

But then it's probably also not a huge deal for it to exist in the context of
the manifest api here.

[no sievers, 2016/08/05 17:55:49]

I should also point out this from the content-api guide
(https://www.chromium.org/developers/content-module/content-api):

"content/public should contain only interfaces, structs and (rarely) static
functions".

Which is of course in this case not a definite recommendation or even rule :)

[pkotwicz, 2016/08/05 18:16:21]

sievers: Do you think that I should move the functions to
third_party/WebKit/public/platform/WebDisplayModeUtil.h and
third_party/WebKit/public/platform/modules/screen_orientation/WebScreenOrientationUtil.h
?

+// Converts a blink::WebDisplayMode to a string. Returns one of
+// https://www.w3.org/TR/appmanifest/#dfn-display-modes-values. Return values
+// are lowercase. Returns an empty string for blink::WebDisplayModeUndefined.
+CONTENT_EXPORT std::string WebDisplayModeToString(
+    blink::WebDisplayMode display);
+
+// Returns the blink::WebDisplayMode which matches |display|.
+// |display| should be one of
+// https://www.w3.org/TR/appmanifest/#dfn-display-modes-values. |display| is
+// case insensitive. Returns blink::WebDisplayModeUndefined if there is no
+// match.
+CONTENT_EXPORT blink::WebDisplayMode WebDisplayModeFromString(
+    const std::string& display);
+
+// Converts a blink::WebScreenOrientationLockType to a string. Returns one of
+// https://www.w3.org/TR/screen-orientation/#orientationlocktype-enum. Return
+// values are lowercase. Returns an empty string for
+// blink::WebScreenOrientationLockDefault.
+CONTENT_EXPORT std::string WebScreenOrientationLockTypeToString(
+    blink::WebScreenOrientationLockType);
+
+// Returns the blink::WebScreenOrientationLockType which matches
+// |orientation|. |orientation| should be one of
+// https://www.w3.org/TR/screen-orientation/#orientationlocktype-enum.
+// |orientation| is case insensitive. Returns
+// blink::WebScreenOrientationLockDefault if there is no match.
+CONTENT_EXPORT blink::WebScreenOrientationLockType
+WebScreenOrientationLockTypeFromString(const std::string& orientation);
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_COMMON_MANIFEST_UTIL_H_