Skip to content
This repository was archived by the owner on Jan 4, 2019. It is now read-only.

Commit 72abdf4

Browse files
authored
Merge pull request #64 from darkdh/favicon_png
Emit png_data for invalid favicon_url of import from HTML
2 parents cc183f2 + e268f15 commit 72abdf4

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

atom/browser/importer/profile_writer.cc

+7-5
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
#include "atom/common/importer/imported_cookie_entry.h"
1515
#include "atom/common/native_mate_converters/string16_converter.h"
1616
#include "atom/common/native_mate_converters/value_converter.h"
17+
#include "base/base64.h"
1718
#include "base/strings/string_number_conversions.h"
1819
#include "base/strings/stringprintf.h"
1920
#include "base/strings/utf_string_conversions.h"
@@ -315,11 +316,12 @@ void ProfileWriter::AddFavicons(
315316
base::DictionaryValue* imported_favicon = new base::DictionaryValue();
316317
imported_favicon->SetString("favicon_url",
317318
favicon.favicon_url.possibly_invalid_spec());
318-
/*
319-
imported_favicon->SetString("png_data",
320-
std::string(favicon.png_data.begin(),
321-
favicon.png_data.end()));
322-
*/
319+
std::string data_url;
320+
data_url.insert(data_url.end(), favicon.png_data.begin(),
321+
favicon.png_data.end());
322+
base::Base64Encode(data_url, &data_url);
323+
data_url.insert(0, "data:image/png;base64,");
324+
imported_favicon->SetString("png_data", data_url);
323325
std::set<GURL>::iterator it;
324326
base::ListValue* urls = new base::ListValue();
325327
for (it = favicon.urls.begin(); it != favicon.urls.end(); ++it) {

0 commit comments

Comments
 (0)