Auto open and close the palette on an eject event.

ash/common/test/test_palette_delegate.h

Index: ash/common/test/test_palette_delegate.h
diff --git a/ash/common/test/test_palette_delegate.h b/ash/common/test/test_palette_delegate.h
index 76fa80e8411f2859199cc0b43dd085499b11a0bd..75c868561b99d72cb713a5203fd41f9eeddf55ba 100644
--- a/ash/common/test/test_palette_delegate.h
+++ b/ash/common/test/test_palette_delegate.h
@@ -30,20 +30,26 @@ class TestPaletteDelegate : public PaletteDelegate {
 
   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);
 };