VRShell: Composing Daydream controller textures from patches

chrome/browser/android/vr_shell/vr_controller_model.h

 // Copyright 2017 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 CHROME_BROWSER_ANDROID_VR_SHELL_VR_CONTROLLER_MODEL_H_
 #define CHROME_BROWSER_ANDROID_VR_SHELL_VR_CONTROLLER_MODEL_H_
 
 #include <memory>
 
 #include "chrome/browser/android/vr_shell/gltf_asset.h"
-#include "third_party/skia/include/core/SkBitmap.h"
+#include "third_party/skia/include/core/SkImage.h"
+#include "ui/gfx/geometry/point.h"
 #include "ui/gl/gl_bindings.h"
 
 namespace vr_shell {
 
 class VrControllerModel {
  public:
   enum State {
     IDLE = 0,
     TOUCHPAD,
     APP,
     SYSTEM,
     // New ControllerStates should be added here, before STATE_COUNT.
     STATE_COUNT,
   };
 
-  // TODO(acondor): Remove these hardcoded paths once VrShell resources
-  // are delivered through component updater.
-  static constexpr char const kComponentName[] = "VrShell";
-  static constexpr char const kDefaultVersion[] = "0";
-
-  static constexpr char const kModelsDirectory[] = "models";
-  static constexpr char const kModelFilename[] = "controller.gltf";
-  static constexpr char const kTexturesDirectory[] = "tex";
-  static constexpr char const* kTextureFilenames[] = {
-      "ddcontroller_idle.png", "ddcontroller_touchpad.png",
-      "ddcontroller_app.png", "ddcontroller_system.png",
+  struct Patch {
+    sk_sp<SkImage> image;
+    gfx::Point offset;
+    Patch();
+    Patch(sk_sp<SkImage> image, const gfx::Point& offset);
+    ~Patch();

[cjgrant, 2017/04/27 18:47:29]

What happens if you just exclude the destructor here?

[acondor_, 2017/04/27 19:01:19]

Chromium codestyle check asked me to provide constructor/destructor for this
struct. I will recheck just in case.

[acondor_, 2017/04/28 14:22:38]

Actually, I don't need this class anymore.

   };
 
   explicit VrControllerModel(
       std::unique_ptr<gltf::Asset> gltf_asset,
       std::vector<std::unique_ptr<gltf::Buffer>> buffers);
   ~VrControllerModel();
 
   const GLvoid* ElementsBuffer() const;
   GLsizeiptr ElementsBufferSize() const;
   const GLvoid* IndicesBuffer() const;
   GLenum DrawMode() const;
   GLsizeiptr IndicesBufferSize() const;
   const gltf::Accessor* IndicesAccessor() const;
   const gltf::Accessor* PositionAccessor() const;
   const gltf::Accessor* TextureCoordinateAccessor() const;
-  void SetTexture(int state, std::unique_ptr<SkBitmap> bitmap);
-  const SkBitmap* GetTexture(int state) const;
+  void SetBaseTexture(sk_sp<SkImage> image);
+  void SetTexturePatch(int state, sk_sp<SkImage> image);
+  sk_sp<SkImage> GetTexture(int state) const;
 
   static std::unique_ptr<VrControllerModel> LoadFromComponent();
 
  private:
   std::unique_ptr<gltf::Asset> gltf_asset_;
-  std::vector<std::unique_ptr<SkBitmap>> texture_bitmaps_;
+  sk_sp<SkImage> base_texture_;
+  sk_sp<SkImage> patches_[STATE_COUNT];
   std::vector<std::unique_ptr<gltf::Buffer>> buffers_;
 
   const char* Buffer() const;
   const gltf::Accessor* Accessor(const std::string& key) const;
 };
 
 }  // namespace vr_shell
 
 #endif  // CHROME_BROWSER_ANDROID_VR_SHELL_VR_CONTROLLER_MODEL_H_