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

Issue 155892: Make Delete history for this day not use a link. (Closed)

Created:
11 years, 5 months ago by arv (Not doing code reviews)
Modified:
9 years, 3 months ago
Reviewers:
Glen Murphy
CC:
chromium-reviews_googlegroups.com, Ben Goodger (Google)
Visibility:
Public.

Description

Use a <button> that looks like a link to prevent right click and middle click. BUG=9591 TEST=Go to the history page and make sure that "Delete history for this day" still works and that it does not allow opening the "link" in a new window. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21239

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -2 lines) Patch
M chrome/browser/resources/history.html View 1 2 2 chunks +8 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
arv (Not doing code reviews)
11 years, 5 months ago (2009-07-21 23:26:21 UTC) #1
Glen Murphy
11 years, 5 months ago (2009-07-21 23:35:32 UTC) #2
nice! LGTM

Powered by Google App Engine
This is Rietveld 408576698