file manager: clean up rename click detection

file manager: clean up rename click detection

This patch changes rename click detection so that a user must slow double click on the *same* file.  The previous code usually required users to click on the same file, but relied on a side effect of the list control (that clicking on a new file resets the selection), which didn't always work.

BUG=chromium-os:16270
TEST=manual testing on chromeos-chrome on linux

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88348

[rginda, 2011-06-07 22:58:14]

http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/file_manager.js (left):

http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/file_manager.js:1841: window.label =
label;
This is some debugging code that slipped in in a previous commit.

http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/mock_chrome.js (right):

http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/mock_chrome.js:188: chrome.test = {
Needed to run harness.html after jamescook's recent checkin.

[achuithb, 2011-06-08 00:57:34]

On 2011/06/07 22:58:14, rginda wrote:
>
http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
> File chrome/browser/resources/file_manager/js/file_manager.js (left):
> 
>
http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
> chrome/browser/resources/file_manager/js/file_manager.js:1841: window.label =
> label;
> This is some debugging code that slipped in in a previous commit.
> 
>
http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
> File chrome/browser/resources/file_manager/js/mock_chrome.js (right):
> 
>
http://codereview.chromium.org/7134004/diff/1/chrome/browser/resources/file_m...
> chrome/browser/resources/file_manager/js/mock_chrome.js:188: chrome.test = {
> Needed to run harness.html after jamescook's recent checkin.


LGTM