mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 2

Do not store resource_path as FilePath

Browse source 
On Windows the path value returned by PathWithoutParams also uses "/" as
separator, so there is no need to use the FilePath class.
This commit is contained in:
Cheng Zhao 2017-02-14 19:55:05 +09:00 • committed by deepak1556
parent cca1db47ee
commit 422b7acece
1 changed files with 3 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

10
atom/browser/ui/webui/pdf_viewer_ui.cc
View file

@ -33,10 +33,7 @@ class BundledDataSource : public content::URLDataSource {
public:
BundledDataSource() {
for (size_t i = 0; i < kPdfViewerResourcesSize; ++i) {
base::FilePath resource_path =
base::FilePath().AppendASCII(kPdfViewerResources[i].name);
resource_path = resource_path.NormalizePathSeparators();
std::string resource_path = kPdfViewerResources[i].name;
DCHECK(path_to_resource_id_.find(resource_path) ==
path_to_resource_id_.end());
path_to_resource_id_[resource_path] = kPdfViewerResources[i].value;
@ -51,8 +48,7 @@ class BundledDataSource : public content::URLDataSource {
const content::ResourceRequestInfo::WebContentsGetter& wc_getter,
const GotDataCallback& callback) override {
std::string filename = PathWithoutParams(path);
auto entry =
path_to_resource_id_.find(base::FilePath::FromUTF8Unsafe(filename));
auto entry = path_to_resource_id_.find(filename);
if (entry != path_to_resource_id_.end()) {
int resource_id = entry->second;
@ -82,7 +78,7 @@ class BundledDataSource : public content::URLDataSource {
~BundledDataSource() override {}
// A map from a resource path to the resource ID.
std::map<base::FilePath, int> path_to_resource_id_;
std::map<std::string, int> path_to_resource_id_;
DISALLOW_COPY_AND_ASSIGN(BundledDataSource);
};