NativeViewport

mojo/services/native_viewport/native_viewport.h

+// Copyright 2013 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 MOJO_SERVICES_NATIVE_VIEWPORT_NATIVE_VIEWPORT_H_
+#define MOJO_SERVICES_NATIVE_VIEWPORT_NATIVE_VIEWPORT_H_
+
+namespace mojo {
+namespace services {
+
+class NativeViewportDelegate;

[abarth-chromium, 2013/10/29 21:48:44]

You don't want to make this an inner class?

+
+// Encapsulation of platform-specific Viewport.
+class NativeViewport {
+ public:
+  virtual ~NativeViewport() {}
+
+  static NativeViewport* CreateNativeViewport(
+      NativeViewportDelegate* delegate);

[abarth-chromium, 2013/10/29 21:48:44]

Sould this return a scoped_ptr<NativeViewport> to force the caller to take
ownership of the returned object?

Also, you can just call this function |Create|.  The "NativeViewport" part of
the name is redundant.

+};
+
+}  // namespace services
+}  // namespace mojo
+
+#endif  // MOJO_SERVICES_NATIVE_VIEWPORT_NATIVE_VIEWPORT_H_