mash: Relay some Chrome aura window properties to Ash.

ui/views/mus/mus_client.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 UI_VIEWS_MUS_MUS_CLIENT_H_
 #define UI_VIEWS_MUS_MUS_CLIENT_H_
 
 #include <stdint.h>
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "ui/aura/mus/window_tree_client_delegate.h"
 #include "ui/base/dragdrop/os_exchange_data_provider_factory.h"
 #include "ui/views/mus/mus_export.h"
+#include "ui/views/mus/mus_property_mirror.h"

[sky, 2017/01/27 20:10:45]

Forward declare?

[msw, 2017/01/27 21:19:25]

Done.

 #include "ui/views/mus/screen_mus_delegate.h"
 #include "ui/views/widget/widget.h"
 
 namespace aura {
 class PropertyConverter;
 class Window;
 class WindowTreeClient;
 }
 
 namespace base {
@@ skipping 23 matching lines @@
 namespace internal {
 class NativeWidgetDelegate;
 }
 
 namespace test {
 class MusClientTestApi;
 }
 
 // MusClient establishes a connection to mus and sets up necessary state so that
 // aura and views target mus. This class is useful for typical clients, not the
-// WindowManager. Most clients don't create this directly, rather use
-// AuraInit.
+// WindowManager. Most clients don't create this directly, rather use AuraInit.
 class VIEWS_MUS_EXPORT MusClient
     : public aura::WindowTreeClientDelegate,
       public ScreenMusDelegate,
       public ui::OSExchangeDataProviderFactory::Factory {
  public:
   // Most clients should use AuraInit, which creates a MusClient.
   // |create_wm_state| indicates whether MusClient should create a wm::WMState.
   MusClient(
       service_manager::Connector* connector,
       const service_manager::Identity& identity,
@@ skipping 24 matching lines @@
 
   // Creates DesktopNativeWidgetAura with DesktopWindowTreeHostMus. This is
   // set as the factory function used for creating NativeWidgets when a
   //  NativeWidget has not been explicitly set.
   NativeWidget* CreateNativeWidget(const Widget::InitParams& init_params,
                                    internal::NativeWidgetDelegate* delegate);
 
   void AddObserver(MusClientObserver* observer);
   void RemoveObserver(MusClientObserver* observer);
 
+  MusPropertyMirror* mus_property_mirror() {
+    return mus_property_mirror_.get();
+  }
+  void set_mus_property_mirror(std::unique_ptr<MusPropertyMirror> mirror) {

[sky, 2017/01/27 20:10:45]

As this runs a destructor I think it should be SetMusPropertyMirror and not
inlined.

[msw, 2017/01/27 21:19:25]

Done.

+    mus_property_mirror_ = std::move(mirror);
+  }
+
  private:
   friend class AuraInit;
   friend class test::MusClientTestApi;
 
   // aura::WindowTreeClientDelegate:
   void OnEmbed(
       std::unique_ptr<aura::WindowTreeHostMus> window_tree_host) override;
   void OnLostConnection(aura::WindowTreeClient* client) override;
   void OnEmbedRootDestroyed(aura::WindowTreeHostMus* window_tree_host) override;
   void OnPointerEventObserved(const ui::PointerEvent& event,
@@ skipping 16 matching lines @@
 
   base::ObserverList<MusClientObserver> observer_list_;
 
   // NOTE: this may be null (creation is based on argument supplied to
   // constructor).
   std::unique_ptr<wm::WMState> wm_state_;
 
   std::unique_ptr<ScreenMus> screen_;
 
   std::unique_ptr<aura::PropertyConverter> property_converter_;
+  std::unique_ptr<MusPropertyMirror> mus_property_mirror_;
 
   std::unique_ptr<aura::WindowTreeClient> window_tree_client_;
 
   std::unique_ptr<PointerWatcherEventRouter> pointer_watcher_event_router_;
 
   std::unique_ptr<discardable_memory::ClientDiscardableSharedMemoryManager>
       discardable_shared_memory_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(MusClient);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_MUS_MUS_CLIENT_H_