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

Unified Diff: src/plugin/PluginClass.cpp

Issue 5427527162003456: Issue 1086 - Add acceptable ads checkbox to the settings page (Closed)
Patch Set: Created July 21, 2014, 11:42 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
@@ -1343,17 +1343,6 @@
}
GetBrowser()->Refresh();
}
- case ID_MENU_ACCEPTABLE_ADS:
- {
- if (client->IsAcceptableAdsEnabled())
- {
- client->RemoveSubscription(client->GetPref(L"subscriptions_exceptionsurl", L""));
- }
- else
- {
- client->AddSubscription(client->GetPref(L"subscriptions_exceptionsurl", L""));
- }
- }
default:
break;
}
@@ -1451,22 +1440,6 @@
fmii.cch = static_cast<UINT>(ctext.size());
::SetMenuItemInfoW(hMenu, ID_MENU_SETTINGS, FALSE, &fmii);
- ctext = dictionary->Lookup("menu", "menu-acceptable-ads");
- if (client->IsAcceptableAdsEnabled())
- {
- fmii.fState = MFS_CHECKED | MFS_ENABLED;
- }
- else
- {
- fmii.fState = MFS_UNCHECKED | MFS_ENABLED;
- }
- fmii.fMask = MIIM_STRING | MIIM_STATE;
- fmii.dwTypeData = const_cast<LPWSTR>(ctext.c_str());
- fmii.cch = static_cast<UINT>(ctext.size());
-
- ::SetMenuItemInfoW(hMenu, ID_MENU_ACCEPTABLE_ADS, FALSE, &fmii);
-
-
return true;
}

Powered by Google App Engine
This is Rietveld