Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/plugin/PluginClass.cpp

Issue 11276031: FRP wrappers. "Update" menu item. (Closed)
Patch Set: GetAppLocale is now retrieved directly from the browser. Get documentation link change. Manual Upda… Created Aug. 5, 2013, 8:03 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/plugin/PluginClass.cpp
===================================================================
--- a/src/plugin/PluginClass.cpp
+++ b/src/plugin/PluginClass.cpp
@@ -1257,6 +1257,12 @@
switch (nCommand)
{
+ case ID_MENU_UPDATE:
+ {
+ CPluginClient* client = CPluginClient::GetInstance();
+ client->CheckForUpdates();
+ }
+ break;
case ID_MENU_DISABLE:
{
CPluginSettings* settings = CPluginSettings::GetInstance();
@@ -1387,12 +1393,12 @@
#endif // SUPPORT_WHITELIST
// Plugin update
- ctext = dictionary->Lookup("menu", "update");
+ ctext = dictionary->Lookup("menu", "menu-update");
Wladimir Palant 2013/08/13 09:19:51 What's the point of prefixing everything in the "m
fmii.fMask = MIIM_STATE | MIIM_STRING;
fmii.fState = client ? MFS_ENABLED : MFS_DISABLED;
fmii.dwTypeData = const_cast<LPWSTR>(ctext.c_str());
fmii.cch = ctext.size();
- ::SetMenuItemInfoW(hMenu, ID_PLUGIN_UPDATE, FALSE, &fmii);
+ ::SetMenuItemInfoW(hMenu, ID_MENU_UPDATE, FALSE, &fmii);
// Plugin enable

Powered by Google App Engine
This is Rietveld