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

Issue 29330709: Issue #3343 - Eliminate unescaping of URL (Closed)

Created:
Nov. 23, 2015, 2:16 p.m. by Eric
Modified:
Nov. 26, 2015, 12:54 p.m.
Reviewers:
sergei, Oleksandr
CC:
Felix Dahlke
Visibility:
Public.

Description

Issue #3343 - Eliminate unescaping of URL Remove UnescapeUrl() and all calls to it.

Patch Set 1 #

Total comments: 3

Patch Set 2 : Reverted unrelated defect introduction #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -46 lines) Patch
M src/plugin/AdblockPlusDomTraverser.cpp View 1 chunk +0 lines, -1 line 0 comments Download
M src/plugin/PluginClass.cpp View 1 3 chunks +0 lines, -3 lines 0 comments Download
M src/plugin/PluginDomTraverserBase.h View 2 chunks +0 lines, -2 lines 0 comments Download
M src/plugin/PluginUtil.h View 1 chunk +0 lines, -10 lines 0 comments Download
M src/plugin/PluginUtil.cpp View 1 chunk +0 lines, -28 lines 0 comments Download
M src/plugin/PluginWbPassThrough.cpp View 1 2 chunks +1 line, -2 lines 0 comments Download

Messages

Total messages: 8
Eric
If we're going to stop unescaping URL's entirely, this is how you do it. I ...
Nov. 23, 2015, 2:18 p.m. (2015-11-23 14:18:51 UTC) #1
Oleksandr
I think this should definitely not be a "Noissue" commit. FYI: https://issues.adblockplus.org/ticket/3343
Nov. 23, 2015, 2:35 p.m. (2015-11-23 14:35:14 UTC) #2
Eric
On 2015/11/23 14:35:14, Oleksandr wrote: > I think this should definitely not be a "Noissue" ...
Nov. 23, 2015, 2:47 p.m. (2015-11-23 14:47:28 UTC) #3
Oleksandr
I don't think we can commit a changeset like this without an appropriate issue. I ...
Nov. 25, 2015, 2:24 a.m. (2015-11-25 02:24:29 UTC) #4
sergei
https://codereview.adblockplus.org/29330709/diff/29330710/src/plugin/PluginClass.cpp File src/plugin/PluginClass.cpp (right): https://codereview.adblockplus.org/29330709/diff/29330710/src/plugin/PluginClass.cpp#newcode490 src/plugin/PluginClass.cpp:490: if (url.find(L"javascript") == std::wstring::npos) On 2015/11/25 02:24:29, Oleksandr wrote: ...
Nov. 25, 2015, 8:29 a.m. (2015-11-25 08:29:14 UTC) #5
Eric
Changed the commit message to issue #3343. https://codereview.adblockplus.org/29330709/diff/29330710/src/plugin/PluginClass.cpp File src/plugin/PluginClass.cpp (right): https://codereview.adblockplus.org/29330709/diff/29330710/src/plugin/PluginClass.cpp#newcode490 src/plugin/PluginClass.cpp:490: if (url.find(L"javascript") ...
Nov. 25, 2015, 4:18 p.m. (2015-11-25 16:18:21 UTC) #6
Oleksandr
LGTM
Nov. 25, 2015, 5:31 p.m. (2015-11-25 17:31:46 UTC) #7
sergei
Nov. 26, 2015, 11:12 a.m. (2015-11-26 11:12:19 UTC) #8
LGTM

Powered by Google App Engine
This is Rietveld