Add create note palette action.

Add create note palette action.

The action is only visible if the default note-taking application is installed. Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

TBR=oshima@chromium.org

BUG=616112
Committed: https://crrev.com/77dfb185da141c1cd558207b62f75cfb895b7f49
Cr-Commit-Position: refs/heads/master@{#413485}

[jdufault, 2016-08-10 22:38:59]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-10 22:39:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 23:34:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-10 23:34:07]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[jdufault, 2016-08-11 00:13:29]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 00:14:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 00:40:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-11 00:40:29]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jdufault, 2016-08-11 19:52:59]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 19:53:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 20:27:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-11 20:27:43]

Dry run: Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[jdufault, 2016-08-11 22:44:00]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 22:44:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 23:17:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-11 23:17:17]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[jdufault, 2016-08-12 20:46:30]

Patchset #1 (id:1) has been deleted

[jdufault, 2016-08-12 20:46:39]

Patchset #1 (id:20001) has been deleted

[jdufault, 2016-08-12 20:46:48]

Patchset #1 (id:40001) has been deleted

[jdufault, 2016-08-12 20:46:59]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 20:47:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 20:51:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-12 20:51:45]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[jdufault, 2016-08-12 21:04:51]

Description was changed from

==========
Add create note palette action.

BUG=616112
==========

to

==========
Add create note palette action.

BUG=616112
==========

[jdufault, 2016-08-12 21:04:51]

jdufault@chromium.org changed reviewers:
+ oshima@chromium.org

[jdufault, 2016-08-12 21:05:01]

oshima@, PTAL. Thanks!

[jdufault, 2016-08-12 22:41:46]

jdufault@chromium.org changed reviewers:
+ jamescook@chromium.org

[jdufault, 2016-08-12 22:42:19]

jamescook@, PTAL. Thanks! I've filed crbug.com/637447 against myself to add
tests.

[James Cook, 2016-08-12 23:58:55]

I would also like to see at least a basic test of this feature. You could add
WmShell::SetPaletteDelegateForTesting() and use it to sense what the ash code is
trying to do.

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tool.cc (right):

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tool.cc:21:
tool_manager->AddTool(base::WrapUnique(new CreateNoteAction(tool_manager)));
MakeUnique

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tool_manager.cc (right):

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tool_manager.cc:82:
views.reserve(tools_.size());
nice attention to detail

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/tools/create_note_action.h (right):

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_action.h:14: class
CreateNoteAction : public CommonPaletteTool {
class doc please

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_action.h:32:
DISALLOW_COPY_AND_ASSIGN(CreateNoteAction);
#include "base/macros.h"

[jdufault, 2016-08-16 21:37:45]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-16 21:38:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jdufault, 2016-08-16 21:38:18]

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tool.cc (right):

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tool.cc:21:
tool_manager->AddTool(base::WrapUnique(new CreateNoteAction(tool_manager)));
On 2016/08/12 23:58:55, James Cook (slow reviews) wrote:
> MakeUnique

Done.

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tool_manager.cc (right):

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tool_manager.cc:82:
views.reserve(tools_.size());
On 2016/08/12 23:58:55, James Cook (slow reviews) wrote:
> nice attention to detail

:)

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/tools/create_note_action.h (right):

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_action.h:14: class
CreateNoteAction : public CommonPaletteTool {
On 2016/08/12 23:58:55, James Cook (slow reviews) wrote:
> class doc please

Done.

https://codereview.chromium.org/2235063002/diff/50001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_action.h:32:
DISALLOW_COPY_AND_ASSIGN(CreateNoteAction);
On 2016/08/12 23:58:55, James Cook (slow reviews) wrote:
> #include "base/macros.h"

Done.

[James Cook, 2016-08-16 22:08:45]

Just nits. Ping me on chat if you need the next patch turned around particularly
quickly.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tool.cc (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tool.cc:15:
tool_manager->AddTool(base::MakeUnique<CreateNoteAction>(tool_manager));
nit: #include base/memory/ptr_util.h for MakeUnique

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/tools/create_note_action.h (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_action.h:8: #include
<memory>
nit: not needed?

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/tools/create_note_unittest.cc (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:15: using
namespace ash;
nit: just wrap the file in

namespace ash {
}

unless there's some reason that won't work

If there are things that can't be in the namespace, put them above the
"namespace ash" block

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:23:
~MockPaletteToolDelegate() {}
override

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:52:
std::unique_ptr<MockPaletteToolDelegate> palette_tool_delegate;
palette_tool_delegate_

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:53:
std::unique_ptr<PaletteTool> tool;
tool_

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
File ash/common/test/test_palette_delegate.h (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:14: class TestPaletteDelegate : public
ash::PaletteDelegate {
nit: no ash

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:26: // ash::PaletteDelegate:
ditto

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:34:
DISALLOW_COPY_AND_ASSIGN(TestPaletteDelegate);
I presume this isn't a local class in the unit test file because you're planning
to use it in other tests?

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:35: };
Hooray for small simple test doubles!

https://codereview.chromium.org/2235063002/diff/70001/ash/common/wm_shell.h
File ash/common/wm_shell.h (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/wm_shell.h#n...
ash/common/wm_shell.h:130: void SetPaletteDelegateForTesting(
nit: Move this down by SetShelfDelegateForTesting

[James Cook, 2016-08-16 22:10:17]

Also, please update the CL description to have a bit more detail and add a TEST=
line.

[jdufault, 2016-08-16 22:43:31]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-16 22:43:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jdufault, 2016-08-16 22:45:38]

Description was changed from

==========
Add create note palette action.

BUG=616112
==========

to

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

BUG=616112
==========

[jdufault, 2016-08-16 22:45:52]

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tool.cc (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tool.cc:15:
tool_manager->AddTool(base::MakeUnique<CreateNoteAction>(tool_manager));
On 2016/08/16 22:08:44, James Cook (slow reviews) wrote:
> nit: #include base/memory/ptr_util.h for MakeUnique

Done.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/tools/create_note_action.h (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_action.h:8: #include
<memory>
On 2016/08/16 22:08:44, James Cook (slow reviews) wrote:
> nit: not needed?

Done.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/tools/create_note_unittest.cc (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:15: using
namespace ash;
On 2016/08/16 22:08:44, James Cook (slow reviews) wrote:
> nit: just wrap the file in
> 
> namespace ash {
> }
> 
> unless there's some reason that won't work
> 
> If there are things that can't be in the namespace, put them above the
> "namespace ash" block

Done.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:23:
~MockPaletteToolDelegate() {}
On 2016/08/16 22:08:45, James Cook (slow reviews) wrote:
> override

Done.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:52:
std::unique_ptr<MockPaletteToolDelegate> palette_tool_delegate;
On 2016/08/16 22:08:44, James Cook (slow reviews) wrote:
> palette_tool_delegate_

Done; moved to protected as well.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/system/chrom...
ash/common/system/chromeos/palette/tools/create_note_unittest.cc:53:
std::unique_ptr<PaletteTool> tool;
On 2016/08/16 22:08:44, James Cook (slow reviews) wrote:
> tool_

Done; moved to protected as well.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
File ash/common/test/test_palette_delegate.h (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:14: class TestPaletteDelegate : public
ash::PaletteDelegate {
On 2016/08/16 22:08:45, James Cook (slow reviews) wrote:
> nit: no ash

Done.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:26: // ash::PaletteDelegate:
On 2016/08/16 22:08:45, James Cook (slow reviews) wrote:
> ditto

Done.

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:34:
DISALLOW_COPY_AND_ASSIGN(TestPaletteDelegate);
On 2016/08/16 22:08:45, James Cook (slow reviews) wrote:
> I presume this isn't a local class in the unit test file because you're
planning
> to use it in other tests?

Yep

https://codereview.chromium.org/2235063002/diff/70001/ash/common/test/test_pa...
ash/common/test/test_palette_delegate.h:35: };
On 2016/08/16 22:08:45, James Cook (slow reviews) wrote:
> Hooray for small simple test doubles!

:)

https://codereview.chromium.org/2235063002/diff/70001/ash/common/wm_shell.h
File ash/common/wm_shell.h (right):

https://codereview.chromium.org/2235063002/diff/70001/ash/common/wm_shell.h#n...
ash/common/wm_shell.h:130: void SetPaletteDelegateForTesting(
On 2016/08/16 22:08:45, James Cook (slow reviews) wrote:
> nit: Move this down by SetShelfDelegateForTesting

Ah, good. I spent some time looking for another testing function but never found
any.

[James Cook, 2016-08-16 22:48:16]

LGTM

[commit-bot: I haz the power, 2016-08-17 00:34:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-17 00:34:09]

Dry run: This issue passed the CQ dry run.

[jdufault, 2016-08-17 19:38:21]

jdufault@chromium.org changed reviewers:
+ estade@chromium.org

[jdufault, 2016-08-17 19:38:40]

estade@ PTAL at ui/*

[jdufault, 2016-08-18 19:51:23]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-18 19:51:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-18 21:03:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-18 21:03:46]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[jdufault, 2016-08-19 19:20:56]

On 2016/08/17 19:38:40, jdufault wrote:
> estade@ PTAL at ui/*

ping

[James Cook, 2016-08-19 20:43:04]

On 2016/08/19 19:20:56, jdufault wrote:
> On 2016/08/17 19:38:40, jdufault wrote:
> > estade@ PTAL at ui/*
> 
> ping

I think estade@ is OOO today. You might need to ping another reviewer.

[jdufault, 2016-08-19 21:25:02]

On 2016/08/19 20:43:04, James Cook (slow reviews) wrote:
> On 2016/08/19 19:20:56, jdufault wrote:
> > On 2016/08/17 19:38:40, jdufault wrote:
> > > estade@ PTAL at ui/*
> > 
> > ping
> 
> I think estade@ is OOO today. You might need to ping another reviewer.

He is the only owner :O. Waiting until Monday is probably okay.

[Evan Stade, 2016-08-22 15:12:21]

lgtm

https://codereview.chromium.org/2235063002/diff/110001/ash/common/system/chro...
File ash/common/system/chromeos/palette/tools/create_note_action.cc (right):

https://codereview.chromium.org/2235063002/diff/110001/ash/common/system/chro...
ash/common/system/chromeos/palette/tools/create_note_action.cc:16: bool
IsNoteAppAvailable() {
why not just inline this in the one place it's used (or even in multiple places
if you use it multiple places)

[jdufault, 2016-08-22 16:29:20]

https://codereview.chromium.org/2235063002/diff/110001/ash/common/system/chro...
File ash/common/system/chromeos/palette/tools/create_note_action.cc (right):

https://codereview.chromium.org/2235063002/diff/110001/ash/common/system/chro...
ash/common/system/chromeos/palette/tools/create_note_action.cc:16: bool
IsNoteAppAvailable() {
On 2016/08/22 15:12:21, Evan Stade wrote:
> why not just inline this in the one place it's used (or even in multiple
places
> if you use it multiple places)

Done.

[jdufault, 2016-08-22 16:29:31]

The CQ bit was checked by jdufault@chromium.org

[jdufault, 2016-08-22 16:29:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamescook@chromium.org, estade@chromium.org
Link to the patchset: https://codereview.chromium.org/2235063002/#ps130001
(title: "Nit")

[commit-bot: I haz the power, 2016-08-22 16:29:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 16:37:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-22 16:37:19]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[jdufault, 2016-08-22 16:41:44]

Description was changed from

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

BUG=616112
==========

to

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

TBR=oshima@chromium.org

BUG=616112
==========

[jdufault, 2016-08-22 16:42:18]

On 2016/08/22 16:37:19, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Added TBR on Oshima for palette_delegate_chromeos.h/cc

[jdufault, 2016-08-22 16:42:23]

The CQ bit was checked by jdufault@chromium.org

[commit-bot: I haz the power, 2016-08-22 16:43:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 18:46:46]

Description was changed from

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

TBR=oshima@chromium.org

BUG=616112
==========

to

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

TBR=oshima@chromium.org

BUG=616112
==========

[commit-bot: I haz the power, 2016-08-22 18:46:47]

Committed patchset #5 (id:130001)

[commit-bot: I haz the power, 2016-08-22 18:48:35]

Description was changed from

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

TBR=oshima@chromium.org

BUG=616112
==========

to

==========
Add create note palette action.

The action is only visible if the default note-taking application is installed.
Activating the action will open up the note-taking app.

TEST=./out/Release/ash_unittests --gtest_filter=CreateNote*

TBR=oshima@chromium.org

BUG=616112

Committed: https://crrev.com/77dfb185da141c1cd558207b62f75cfb895b7f49
Cr-Commit-Position: refs/heads/master@{#413485}
==========

[commit-bot: I haz the power, 2016-08-22 18:48:40]

Patchset 5 (id:??) landed as
https://crrev.com/77dfb185da141c1cd558207b62f75cfb895b7f49
Cr-Commit-Position: refs/heads/master@{#413485}