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

Side by Side Diff: src/plugin/PluginClass.cpp

Issue 4920381332783104: Issue 1339 - Page automatically reloads when ABP enabled or disabled by user (Closed)
Patch Set: Created March 27, 2015, 8:11 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2015 Eyeo GmbH 3 * Copyright (C) 2006-2015 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 1142 matching lines...) Expand 10 before | Expand all | Expand 10 after
1153 CPluginSettings* settings = CPluginSettings::GetInstance(); 1153 CPluginSettings* settings = CPluginSettings::GetInstance();
1154 std::wstring urlString = GetTab()->GetDocumentUrl(); 1154 std::wstring urlString = GetTab()->GetDocumentUrl();
1155 if (client->IsWhitelistedUrl(urlString)) 1155 if (client->IsWhitelistedUrl(urlString))
1156 { 1156 {
1157 settings->RemoveWhiteListedDomain(ToCString(client->GetHostFromUrl(urlSt ring))); 1157 settings->RemoveWhiteListedDomain(ToCString(client->GetHostFromUrl(urlSt ring)));
1158 } 1158 }
1159 else 1159 else
1160 { 1160 {
1161 settings->AddWhiteListedDomain(ToCString(client->GetHostFromUrl(urlStrin g))); 1161 settings->AddWhiteListedDomain(ToCString(client->GetHostFromUrl(urlStrin g)));
1162 } 1162 }
1163 GetBrowser()->Refresh();
Eric 2015/03/27 13:56:23 When a user disables ABP, blocked items (if any) s
Oleksandr 2015/03/27 14:53:59 Why should they? They will reappear after the refr
1164 } 1163 }
1165 default: 1164 default:
1166 break; 1165 break;
1167 } 1166 }
1168 1167
1169 // Invalidate and redraw the control 1168 // Invalidate and redraw the control
1170 UpdateStatusBar(); 1169 UpdateStatusBar();
1171 } 1170 }
1172 1171
1173 1172
(...skipping 583 matching lines...) Expand 10 before | Expand all | Expand 10 after
1757 s_criticalSectionLocal.Unlock(); 1756 s_criticalSectionLocal.Unlock();
1758 1757
1759 } 1758 }
1760 } 1759 }
1761 } 1760 }
1762 1761
1763 hTabWnd = ::GetWindow(hTabWnd, GW_HWNDNEXT); 1762 hTabWnd = ::GetWindow(hTabWnd, GW_HWNDNEXT);
1764 } 1763 }
1765 return hTabWnd; 1764 return hTabWnd;
1766 } 1765 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld