Comment #5 on issue 17831 by [email protected]: Download shelf item  
layout is too dumb
http://code.google.com/p/chromium/issues/detail?id=17831

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=23103

------------------------------------------------------------------------
r23103 | [email protected] | 2009-08-11 15:50:13 -0700 (Tue, 11 Aug 2009)  
| 6 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/nibs/DownloadItem.xib?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/download_item_controller.h?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/download_item_controller.mm?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/download_item_mac.h?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/download_item_mac.mm?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/download_shelf_controller.h?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/download_shelf_controller.mm?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/download_exe.cc?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/download_manager.cc?r1=23103&r2=23102
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.gyp?r1=23103&r2=23102

Add "dangerous download" view on OS X, for now for dmg files. Also fix  
download item layout.

BUG=14667,17831
TEST=Download a dmg file. Make sure the "dangerous download" view appears.  
Check that both buttons work, and that the dmg filename is elided  
correctly. Also test that "Remove" on a normal download item's context menu  
relayouts the remaining items on the shelf.

Review URL: http://codereview.chromium.org/165295
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to