Chromium Code Reviews

Issue 5685008: Cookies tree js code touch up. (Closed)

Created:
10 years ago by xiyuan
Modified:
9 years, 7 months ago
Reviewers:
arv (Not doing code reviews)
CC:
chromium-reviews, arv (Not doing code reviews)
Visibility:
Public.

Description

Cookies tree js code touch up. BUG=none. TEST=No behavior change. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=69081

Patch Set 1 #

Total comments: 2

Patch Set 2 : address arv's comments #

Unified diffs Side-by-side diffs Stats (+16 lines, -5 lines)
M chrome/browser/resources/options/cookies_view.js View 4 chunks +16 lines, -5 lines 0 comments

Messages

Total messages: 3 (0 generated)
arv (Not doing code reviews)
http://codereview.chromium.org/5685008/diff/1/chrome/browser/resources/options/cookies_view.js File chrome/browser/resources/options/cookies_view.js (right): http://codereview.chromium.org/5685008/diff/1/chrome/browser/resources/options/cookies_view.js#newcode257 chrome/browser/resources/options/cookies_view.js:257: // on Mac. Then you can remove the stopPropagation.
10 years ago (2010-12-13 20:57:29 UTC) #1
xiyuan
http://codereview.chromium.org/5685008/diff/1/chrome/browser/resources/options/cookies_view.js File chrome/browser/resources/options/cookies_view.js (right): http://codereview.chromium.org/5685008/diff/1/chrome/browser/resources/options/cookies_view.js#newcode257 chrome/browser/resources/options/cookies_view.js:257: // on Mac. On 2010/12/13 20:57:29, arv wrote: > ...
10 years ago (2010-12-13 21:12:33 UTC) #2
arv (Not doing code reviews)
10 years ago (2010-12-13 21:31:41 UTC) #3 
LGTM

Powered by Google App Engine