Marco Bonardo [:mak] Assignee
	Description • 10 years ago

it's currently using getURIForKeyword to check if the user is inserting a search keyword duplicate of a bookmarks keyword.

The problem is that editKeyword is synchronous, but I guess we could make it async, since it's only used by a command.

(honestly I'm not sure if that still makes sense, we could just decide a keywords priority in unified complete and let the user create dupes...)

	Marco Bonardo [:mak] Assignee
	Comment 1 • 9 years ago

/browser/components/search/content/engineManager.js (View Hg log or Hg annotations)

    line 127 -- if (bmserv.getURIForKeyword(alias.value))

see also bug 1148457

	Marco Bonardo [:mak] Assignee
	Comment 2 • 9 years ago

This was easier than expected

	Marco Bonardo [:mak] Assignee
	Comment 3 • 9 years ago

Note, I tested this manually with in-content prefs, old prefs and old search ui.

	Marco Bonardo [:mak] Assignee
	Comment 4 • 9 years ago

Comment on attachment 8590863 [details] [diff] [review]
patch v1

Review of attachment 8590863 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/components/search/content/engineManager.xul
@@ +35,5 @@
>      <command id="cmd_movedown"
>               oncommand="gEngineManagerDialog.bump(-1);"
>               disabled="true"/>
>      <command id="cmd_editkeyword"
> +             oncommand="gEngineManagerDialog.editKeyword().catch(Cu.reportError);"

self-comment: I should use Components.utils rather than Cu.

	Pulsebot
	Comment 5 • 9 years ago

https://hg.mozilla.org/integration/fx-team/rev/75afd4419cd3

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/75afd4419cd3