ozone: Introduce ClientPixmap and ClientPixmapFactory for non-GPU processes.

ui/ozone/common/stub_client_native_pixmap_manager.h

+// Copyright 2015 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 UI_OZONE_COMMON_STUB_CLIENT_NATIVE_PIXMAP_MANAGER_H_
+#define UI_OZONE_COMMON_STUB_CLIENT_NATIVE_PIXMAP_MANAGER_H_
+
+#include "ui/ozone/public/client_native_pixmap_manager.h"
+
+namespace ui {
+
+// Platforms which don't need to share native pixmap use this.
+// The caller takes ownership of the instance.

[dcheng, 2015/07/31 18:09:52]

Returning a scoped_ptr<T> here would be a nice way to express that =)

[dshwang, 2015/07/31 18:59:29]

It's used in "ClientNativePixmapManager*
CreateClientNativePixmapManager<Platform>()" function, which is generated by
script.
If it returns scoped_ptr<T>, the function will become a bit weird.
ClientNativePixmapManager* CreateClientNativePixmapManager<Platform>() {
  return CreateStubClientNativePixmapManager().release();
}

It's nice suggestion. I'll change all of them with script change in another CL.

[dcheng, 2015/07/31 19:02:39]

Why not just do it in this patch, since it's introducing it?

[spang, 2015/08/01 02:14:56]

Because it will domino to all the places this is called, and those places have a
defined signature (to match some generated code). I'd prefer to avoid the churn
and leave it alone personally.

It does get wrapped in a scoped_ptr eventually, always (at the call to
make_scoped_ptr() in ui/ozone/platform_object_internal.h).

[dshwang, 2015/08/03 12:51:48]

Thank you for explanation.

+ClientNativePixmapManager* CreateStubClientNativePixmapManager();
+
+}  // namespace ui
+
+#endif  // UI_OZONE_COMMON_STUB_CLIENT_NATIVE_PIXMAP_MANAGER_H_