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

Side by Side Diff: chrome/browser/resources/active_downloads.html

Issue 7324012: Escape file path in file:// url (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 9 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE HTML> 1 <!DOCTYPE HTML>
2 <html i18n-values="dir:textdirection;"> 2 <html i18n-values="dir:textdirection;">
3 <head> 3 <head>
4 <meta charset="utf-8"> 4 <meta charset="utf-8">
5 <title i18n-content="title"></title> 5 <title i18n-content="title"></title>
6 <link rel="icon" href="../../app/theme/downloads_favicon.png"> 6 <link rel="icon" href="../../app/theme/downloads_favicon.png">
7 <style type="text/css"> 7 <style type="text/css">
8 *:-khtml-drag { 8 *:-khtml-drag {
9 background-color: rgba(238,238,238, 0.5); 9 background-color: rgba(238,238,238, 0.5);
10 } 10 }
(...skipping 401 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 412
413 setErrorText_: function(text) { 413 setErrorText_: function(text) {
414 this.filename.textContent = text; 414 this.filename.textContent = text;
415 }, 415 },
416 416
417 supportsPdf_: function() { 417 supportsPdf_: function() {
418 return 'application/pdf' in navigator.mimeTypes; 418 return 'application/pdf' in navigator.mimeTypes;
419 }, 419 },
420 420
421 openFilePath_: function() { 421 openFilePath_: function() {
422 chromeSend('openNewFullWindow', 'file://' + this.path); 422 chromeSend('openNewFullWindow', 'file://' + escape(this.path));
423 }, 423 },
424 424
425 /** 425 /**
426 * Determine onclick behavior based on filename. 426 * Determine onclick behavior based on filename.
427 * @private 427 * @private
428 */ 428 */
429 getFunctionForItem_: function() { 429 getFunctionForItem_: function() {
430 var path = this.path; 430 var path = this.path;
431 var self = this; 431 var self = this;
432 432
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
857 <body onload="init();" onclick="menu.clear();" onselectstart="return false;" 857 <body onload="init();" onclick="menu.clear();" onselectstart="return false;"
858 i18n-values=".style.fontFamily:fontfamily" oncontextmenu="return false;" 858 i18n-values=".style.fontFamily:fontfamily" oncontextmenu="return false;"
859 onBlur="menu.clear();"> 859 onBlur="menu.clear();">
860 <div id="main" class="columnlist"></div> 860 <div id="main" class="columnlist"></div>
861 <div id="showalldownloads" class="showalldownloads"> 861 <div id="showalldownloads" class="showalldownloads">
862 <span id="showalldownloadstext" class="showalldownloadstext" 862 <span id="showalldownloadstext" class="showalldownloadstext"
863 onclick="showAllDownloads()"></span> 863 onclick="showAllDownloads()"></span>
864 </div> 864 </div>
865 </body> 865 </body>
866 </html> 866 </html>
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698