mash: Fix mash app shelf items; encapsulate ash property setup.

chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc

Index: chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
diff --git a/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc b/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
index aab4bb8588fd54a98d1a620d393182338bc268c9..314a25aaddd94557f2b4313d521d8a6439aaf3cf 100644
--- a/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
+++ b/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
@@ -4,10 +4,7 @@
 
 #include "chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.h"
 
-#include "ash/public/cpp/mus_property_mirror_ash.h"
-#include "ash/public/cpp/window_pin_type.h"
 #include "ash/public/cpp/window_properties.h"
-#include "ash/public/interfaces/window_pin_type.mojom.h"
 #include "ash/root_window_controller.h"
 #include "ash/shelf/shelf_model.h"
 #include "ash/shell.h"
@@ -30,11 +27,8 @@
 #include "chrome/browser/ui/views/frame/immersive_handler_factory_mus.h"
 #include "chrome/browser/ui/views/select_file_dialog_extension.h"
 #include "chrome/browser/ui/views/select_file_dialog_extension_factory.h"
-#include "ui/aura/mus/property_converter.h"
-#include "ui/base/class_property.h"
 #include "ui/keyboard/content/keyboard.h"
 #include "ui/keyboard/keyboard_controller.h"
-#include "ui/views/mus/mus_client.h"
 
 ChromeBrowserMainExtraPartsAsh::ChromeBrowserMainExtraPartsAsh() {}
 
@@ -46,29 +40,7 @@ void ChromeBrowserMainExtraPartsAsh::ServiceManagerConnectionStarted(
     // ash::Shell will not be created because ash is running out-of-process.
     ash::Shell::SetIsBrowserProcessWithMash();
 
-    // Register ash-specific window properties with Chrome's property converter.
-    // This propagates ash properties set on chrome windows to ash, via mojo.
-    DCHECK(views::MusClient::Exists());
-    views::MusClient* mus_client = views::MusClient::Get();
-    aura::WindowTreeClientDelegate* delegate = mus_client;
-    aura::PropertyConverter* converter = delegate->GetPropertyConverter();
-
-    converter->RegisterPrimitiveProperty(
-        ash::kPanelAttachedKey,
-        ui::mojom::WindowManager::kPanelAttached_Property,
-        aura::PropertyConverter::CreateAcceptAnyValueCallback());
-    converter->RegisterPrimitiveProperty(
-        ash::kShelfItemTypeKey,
-        ui::mojom::WindowManager::kShelfItemType_Property,
-        base::Bind(&ash::IsValidShelfItemType));
-    converter->RegisterPrimitiveProperty(
-        ash::kWindowPinTypeKey, ash::mojom::kWindowPinType_Property,
-        base::Bind(&ash::IsValidWindowPinType));
-    converter->RegisterStringProperty(
-        ash::kShelfIDKey, ui::mojom::WindowManager::kShelfID_Property);

[sky, 2017/06/02 13:36:39]

Ideally clients would not have to do anything special to have their windows
appear in the shelf. What would it take to make that happen? Say for example
having task viewers window appear in the shelf without having task view
explicitly set the type and shelf id.

[msw, 2017/06/02 15:25:15]

We would need some way to (a) disambiguate windows that should be shown in the
shelf, like task_viewer, and windows that shouldn't, like chrome's 'restore
pages?' bubble (which is apparently a top-level window, perhaps with a transient
parent? maybe filter by aura::Window::type?), and (b) we'd need to do something
like my old ShelfWindowWatcher hack that assigned arbitrary types and ids to
windows before (or in lieu) of Chrome/app doing so, then fix that up once
Chrome/app assigns an id and a type, I'd essentially be restoring the code i
removed here:
https://codereview.chromium.org/2833173002/diff/340001/ash/shelf/shelf_window...

-
-    mus_client->SetMusPropertyMirror(
-        base::MakeUnique<ash::MusPropertyMirrorAsh>());
+    ash::RegisterWindowPropertiesForTransportAndMirroring();
   }
 }