Converts ash/wm/mru_window_tracker_unittest.cc to use AshTest

ash/mus/property_util.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 ASH_MUS_PROPERTY_UTIL_H_
 #define ASH_MUS_PROPERTY_UTIL_H_
 
 #include "ash/public/interfaces/ash_window_type.mojom.h"
 #include "ash/public/interfaces/container.mojom.h"
 #include "services/ui/public/cpp/window.h"
@@ skipping 70 matching lines @@
 
 bool ShouldRemoveStandardFrame(ui::Window* window);
 
 // See description of |WindowManager::kRendererParentTitleArea_Property|.
 bool ShouldRenderParentTitleArea(ui::Window* window);
 
 // Returns the kInitialDisplayId_Property if present, otherwise
 // kInvalidDisplayID.
 int64_t GetInitialDisplayId(const ui::Window::SharedProperties& properties);
 
+// Manipulates the kExcludeFromMru_Property property.

[msw, 2016/10/11 22:36:05]

optional nit: combine comments, remove blank line between.

[sky, 2016/10/11 22:57:05]

I like keeping separate as that way the comments are more focused.

+void SetExcludeFromMru(ui::Window* window, bool value);

[msw, 2016/10/11 22:36:05]

aside: it'd be nice if these were in a window_properties namespace or similar.

[sky, 2016/10/11 22:57:05]

How come? Generally we don't use namespaces based on files, but we've definitely
waffled on that.

+
+// Returns true if the property is set and true, otherwise false.
+bool GetExcludeFromMru(const ui::Window* window);

[msw, 2016/10/11 22:36:05]

very optional nit: IsExcludedFromMru? (like Is* bool getters above).

[sky, 2016/10/11 22:57:05]

I went with this to match the name of the property (both mus and ash one).

+
 }  // namespace mus
 }  // namespace ash
 
 #endif  // ASH_MUS_PROPERTY_UTIL_H_