Separate out Touch Selection Orientation enum

content/browser/renderer_host/input/touch_handle_orientation.h

+// Copyright 2014 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_BROWSER_RENDERER_HOST_INPUT_TOUCH_HANDLE_ORIENTATION_
+#define CONTENT_BROWSER_RENDERER_HOST_INPUT_TOUCH_HANDLE_ORIENTATION_
+
+namespace content {
+
+// This file contains a list of Orientation types for Touch handles, used

[jdduke (slow), 2014/11/06 16:41:59]

Let's remove the text "This file contains a list of ", reformatting the comment
accordingly.

[AviD, 2014/11/10 12:13:48]

Done.

+// for setting the type of handle orientation on java and native side.
+//
+// A Java counterpart will be generated for this enum.
+// GENERATED_JAVA_ENUM_PACKAGE: org.chromium.content.browser.input
+

[jdduke (slow), 2014/11/06 16:41:59]

Nit: Remove line break between comment and enum.

Also, can you add a comment comment line below // GENERATED_JAVA_ENUM_PACKAGE...
that goes:

// GENERATED_JAVA_PREFIX_TO_STRIP: TOUCH_HANDLE_

Then you can update the Java side to use a shorter name :)

[AviD, 2014/11/10 12:13:48]

Done.

+enum TouchHandleOrientation {
+  TOUCH_HANDLE_LEFT,
+  TOUCH_HANDLE_CENTER,
+  TOUCH_HANDLE_RIGHT,
+  TOUCH_HANDLE_ORIENTATION_UNDEFINED,
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_RENDERER_HOST_INPUT_TOUCH_HANDLE_ORIENTATION_