Auto open and close the palette on an eject event.

ash/common/test/test_palette_delegate.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 ASH_COMMON_TEST_TEST_PALETTE_DELEGATE_H_
 #define ASH_COMMON_TEST_TEST_PALETTE_DELEGATE_H_
 
 #include "ash/common/palette_delegate.h"
 #include "base/macros.h"
 
@@ skipping 12 matching lines @@
   bool partial_magnifier_state() const { return partial_magnifier_state_; }
 
   int take_screenshot_count() const { return take_screenshot_count_; }
 
   int take_partial_screenshot_count() const {
     return take_partial_screenshot_count_;
   }
 
   void set_has_note_app(bool has_note_app) { has_note_app_ = has_note_app; }
 
+  void set_should_auto_open_palette(bool should_auto_open_palette) {
+    should_auto_open_palette_ = should_auto_open_palette;
+  }
+
  private:
   // PaletteDelegate:
   void CreateNote() override;
   bool HasNoteApp() override;
   void SetPartialMagnifierState(bool enabled) override;
+  bool ShouldAutoOpenPalette() override;

[oshima, 2016/08/24 23:38:50]

optional: can this be const method?

[jdufault, 2016/08/24 23:54:07]

Leaving as non-const for the time being to stay parallel with HasNoteApp().
HasNoteApp needs to be non-const for TestPaletteDelegate. I want to convert
TestPaletteDelegate to be an actual mock class, which should at that point allow
for conversion to const methods.

   void TakeScreenshot() override;
   void TakePartialScreenshot() override;
 
-  bool has_note_app_ = false;
   int create_note_count_ = 0;
   int has_note_app_count_ = 0;
   bool partial_magnifier_state_ = false;
   int take_screenshot_count_ = 0;
   int take_partial_screenshot_count_ = 0;
+  bool has_note_app_ = false;
+  bool should_auto_open_palette_ = false;
 
   DISALLOW_COPY_AND_ASSIGN(TestPaletteDelegate);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_TEST_TEST_PALETTE_DELEGATE_H_