tapted
2016/12/19 07:41:21
ui/base can't depend on chrome. I think we will ne
ui/base can't depend on chrome. I think we will need to pass this in via the
delegate interface.
Perhaps IsLookUpAvailable() can take an int* for the command_id to use.
Then I think it's OK to keep using the IDS_ versions of the other commands (i.e.
those that appear in submenus rather than in the root level menu).
However, we can't use `IsTextServicesCommandId` the same way, since we can't
guarantee there's no overlap. However, this shouldn't be a big problem, since
it's just the lookup command that the root menu should see.
spqchan
2016/12/21 01:03:02
That works too. I think we should just take lookup
On 2016/12/19 07:41:21, tapted wrote:
> ui/base can't depend on chrome. I think we will need to pass this in via the
> delegate interface.
>
> Perhaps IsLookUpAvailable() can take an int* for the command_id to use.
>
> Then I think it's OK to keep using the IDS_ versions of the other commands
(i.e.
> those that appear in submenus rather than in the root level menu).
>
> However, we can't use `IsTextServicesCommandId` the same way, since we can't
> guarantee there's no overlap. However, this shouldn't be a big problem, since
> it's just the lookup command that the root menu should see.
That works too. I think we should just take lookup out completely, since the
code would be a bit cleaner too
Issue 2164483006: [MacViews] Implemented text context menu
(Closed)
Created 4 years, 5 months ago by spqchan
Modified 3 years, 10 months ago
Reviewers: tapted, msw, Alexei Svitkine (slow), nasko, sky
Base URL: https://chromium.googlesource.com/chromium/src.git@master
Comments: 373