Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Side by Side Diff: chrome/browser/shell_integration_linux.cc

Issue 12881003: ShortcutInfo::favicon is now a gfx::ImageFamily instead of gfx::Image. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: web_app_mac: Fix compile error (no Mac to test it on so I'm building in the dark). Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/shell_integration_linux.h" 5 #include "chrome/browser/shell_integration_linux.h"
6 6
7 #include <fcntl.h> 7 #include <fcntl.h>
8 #include <glib.h> 8 #include <glib.h>
9 #include <stdlib.h> 9 #include <stdlib.h>
10 #include <sys/stat.h> 10 #include <sys/stat.h>
(...skipping 18 matching lines...) Expand all
29 #include "base/strings/string_tokenizer.h" 29 #include "base/strings/string_tokenizer.h"
30 #include "base/threading/thread.h" 30 #include "base/threading/thread.h"
31 #include "base/threading/thread_restrictions.h" 31 #include "base/threading/thread_restrictions.h"
32 #include "base/utf_string_conversions.h" 32 #include "base/utf_string_conversions.h"
33 #include "build/build_config.h" 33 #include "build/build_config.h"
34 #include "chrome/browser/web_applications/web_app.h" 34 #include "chrome/browser/web_applications/web_app.h"
35 #include "chrome/common/chrome_constants.h" 35 #include "chrome/common/chrome_constants.h"
36 #include "content/public/browser/browser_thread.h" 36 #include "content/public/browser/browser_thread.h"
37 #include "googleurl/src/gurl.h" 37 #include "googleurl/src/gurl.h"
38 #include "ui/gfx/codec/png_codec.h" 38 #include "ui/gfx/codec/png_codec.h"
39 #include "ui/gfx/image/image_skia.h" 39 #include "ui/gfx/icon_family.h"
40 #include "ui/gfx/image/image_skia_rep.h"
41 40
42 using content::BrowserThread; 41 using content::BrowserThread;
43 42
44 namespace { 43 namespace {
45 44
46 // Helper to launch xdg scripts. We don't want them to ask any questions on the 45 // Helper to launch xdg scripts. We don't want them to ask any questions on the
47 // terminal etc. The function returns true if the utility launches and exits 46 // terminal etc. The function returns true if the utility launches and exits
48 // cleanly, in which case |exit_code| returns the utility's exit code. 47 // cleanly, in which case |exit_code| returns the utility's exit code.
49 bool LaunchXdgUtility(const std::vector<std::string>& argv, int* exit_code) { 48 bool LaunchXdgUtility(const std::vector<std::string>& argv, int* exit_code) {
50 // xdg-settings internally runs xdg-mime, which uses mv to move newly-created 49 // xdg-settings internally runs xdg-mime, which uses mv to move newly-created
(...skipping 17 matching lines...) Expand all
68 return false; 67 return false;
69 } 68 }
70 close(devnull); 69 close(devnull);
71 70
72 return base::WaitForExitCode(handle, exit_code); 71 return base::WaitForExitCode(handle, exit_code);
73 } 72 }
74 73
75 std::string CreateShortcutIcon( 74 std::string CreateShortcutIcon(
76 const ShellIntegration::ShortcutInfo& shortcut_info, 75 const ShellIntegration::ShortcutInfo& shortcut_info,
77 const base::FilePath& shortcut_filename) { 76 const base::FilePath& shortcut_filename) {
78 if (shortcut_info.favicon.IsEmpty()) 77 if (shortcut_info.favicon.empty())
79 return std::string(); 78 return std::string();
80 79
81 // TODO(phajdan.jr): Report errors from this function, possibly as infobars. 80 // TODO(phajdan.jr): Report errors from this function, possibly as infobars.
82 base::ScopedTempDir temp_dir; 81 base::ScopedTempDir temp_dir;
83 if (!temp_dir.CreateUniqueTempDir()) 82 if (!temp_dir.CreateUniqueTempDir())
84 return std::string(); 83 return std::string();
85 84
86 base::FilePath temp_file_path = temp_dir.path().Append( 85 base::FilePath temp_file_path = temp_dir.path().Append(
87 shortcut_filename.ReplaceExtension("png")); 86 shortcut_filename.ReplaceExtension("png"));
88 std::string icon_name = temp_file_path.BaseName().RemoveExtension().value(); 87 std::string icon_name = temp_file_path.BaseName().RemoveExtension().value();
89 88
90 std::vector<gfx::ImageSkiaRep> image_reps = 89 for (gfx::IconFamily::const_iterator it = shortcut_info.favicon.begin();
91 shortcut_info.favicon.ToImageSkia()->image_reps(); 90 it != shortcut_info.favicon.end(); ++it) {
92 for (std::vector<gfx::ImageSkiaRep>::const_iterator it = image_reps.begin();
93 it != image_reps.end(); ++it) {
94 std::vector<unsigned char> png_data; 91 std::vector<unsigned char> png_data;
95 const SkBitmap& bitmap = it->sk_bitmap(); 92 int size = it->first;
93 const SkBitmap& bitmap = *it->second.bitmap();
96 if (!gfx::PNGCodec::EncodeBGRASkBitmap(bitmap, false, &png_data)) { 94 if (!gfx::PNGCodec::EncodeBGRASkBitmap(bitmap, false, &png_data)) {
97 // If the bitmap could not be encoded to PNG format, skip it. 95 // If the bitmap could not be encoded to PNG format, skip it.
98 LOG(WARNING) << "Could not encode icon " << icon_name << ".png at size " 96 LOG(WARNING) << "Could not encode icon " << icon_name << ".png at size "
99 << bitmap.width() << "."; 97 << size << ".";
100 continue; 98 continue;
101 } 99 }
102 int bytes_written = file_util::WriteFile(temp_file_path, 100 int bytes_written = file_util::WriteFile(temp_file_path,
103 reinterpret_cast<char*>(png_data.data()), png_data.size()); 101 reinterpret_cast<char*>(png_data.data()), png_data.size());
104 102
105 if (bytes_written != static_cast<int>(png_data.size())) 103 if (bytes_written != static_cast<int>(png_data.size()))
106 return std::string(); 104 return std::string();
107 105
108 std::vector<std::string> argv; 106 std::vector<std::string> argv;
109 argv.push_back("xdg-icon-resource"); 107 argv.push_back("xdg-icon-resource");
110 argv.push_back("install"); 108 argv.push_back("install");
111 109
112 // Always install in user mode, even if someone runs the browser as root 110 // Always install in user mode, even if someone runs the browser as root
113 // (people do that). 111 // (people do that).
114 argv.push_back("--mode"); 112 argv.push_back("--mode");
115 argv.push_back("user"); 113 argv.push_back("user");
116 114
117 argv.push_back("--size"); 115 argv.push_back("--size");
118 argv.push_back(base::IntToString(bitmap.width())); 116 argv.push_back(base::IntToString(size));
119 117
120 argv.push_back(temp_file_path.value()); 118 argv.push_back(temp_file_path.value());
121 argv.push_back(icon_name); 119 argv.push_back(icon_name);
122 int exit_code; 120 int exit_code;
123 if (!LaunchXdgUtility(argv, &exit_code) || exit_code) { 121 if (!LaunchXdgUtility(argv, &exit_code) || exit_code) {
124 LOG(WARNING) << "Could not install icon " << icon_name << ".png at size " 122 LOG(WARNING) << "Could not install icon " << icon_name << ".png at size "
125 << bitmap.width() << "."; 123 << size << ".";
126 } 124 }
127 } 125 }
128 return icon_name; 126 return icon_name;
129 } 127 }
130 128
131 bool CreateShortcutOnDesktop(const base::FilePath& shortcut_filename, 129 bool CreateShortcutOnDesktop(const base::FilePath& shortcut_filename,
132 const std::string& contents) { 130 const std::string& contents) {
133 // Make sure that we will later call openat in a secure way. 131 // Make sure that we will later call openat in a secure way.
134 DCHECK_EQ(shortcut_filename.BaseName().value(), shortcut_filename.value()); 132 DCHECK_EQ(shortcut_filename.BaseName().value(), shortcut_filename.value());
135 133
(...skipping 703 matching lines...) Expand 10 before | Expand all | Expand 10 after
839 837
840 base::FilePath shortcut_filename = GetExtensionShortcutFilename( 838 base::FilePath shortcut_filename = GetExtensionShortcutFilename(
841 profile_path, extension_id); 839 profile_path, extension_id);
842 DCHECK(!shortcut_filename.empty()); 840 DCHECK(!shortcut_filename.empty());
843 841
844 DeleteShortcutOnDesktop(shortcut_filename); 842 DeleteShortcutOnDesktop(shortcut_filename);
845 DeleteShortcutInApplicationsMenu(shortcut_filename); 843 DeleteShortcutInApplicationsMenu(shortcut_filename);
846 } 844 }
847 845
848 } // namespace ShellIntegrationLinux 846 } // namespace ShellIntegrationLinux
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698