Issue 4395 - Setting page is loaded non-populated on some systems

src/shared/Utils.cpp

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 #include <memory>
 #include <stdexcept>
 #include <vector>
 
 #include <Windows.h>
 #include <ShlObj.h>
+#include <Shlwapi.h>
 
 #include "Utils.h"
 
 namespace
 {
   // See http://blogs.msdn.com/b/oldnewthing/archive/2004/10/25/247180.aspx
   EXTERN_C IMAGE_DOS_HEADER __ImageBase;
 
   std::wstring appDataPath;
 
@@ skipping 163 matching lines @@
     return L"";
   }
   auto queryStringBeginsAt = questionSignPos + 1;
   auto endQueryStringPos = url.find(L'#', queryStringBeginsAt);
   if (endQueryStringPos == std::wstring::npos)
   {
     endQueryStringPos = url.length();
   }
   return url.substr(queryStringBeginsAt, endQueryStringPos - queryStringBeginsAt);
 }
+
+std::wstring CanonicalizeUrl(const std::wstring& url)
+{
+  std::wstring urlCanonicalized;
+  urlCanonicalized.resize(url.length() * 2);
+  DWORD urlSize = urlCanonicalized.length();
+  UrlCanonicalizeW(url.c_str(), &urlCanonicalized[0], &urlSize, NULL); 

[sergei, 2016/09/07 08:23:31]

What about using of 0 instead of NULL for dwFlags here and in UrlEscape? NULL is
usually used for pointers but here the argument is a DWORD mask.

[Oleksandr, 2016/09/07 09:13:43]

Done.

+  urlCanonicalized.resize(urlSize);
+  return urlCanonicalized;
+}
+
+std::wstring EscapeUrl(const std::wstring& url)
+{
+  std::wstring urlEscaped;
+  urlEscaped.resize(url.length() * 2);
+  DWORD urlSize = urlEscaped.length();
+  UrlEscapeW(url.c_str(), &urlEscaped[0], &urlSize, NULL); 
+  urlEscaped.resize(urlSize);
+  return urlEscaped;
+}
+