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

Issue 12377095: Add "clear" to about://instant. (Closed)

Created:
7 years, 9 months ago by samarth
Modified:
7 years, 9 months ago
Reviewers:
sreeram, Evan Stade
CC:
chromium-reviews, melevin, sreeram, gideonwald, dominich, David Black, arv+watch_chromium.org, samarth+watch_chromium.org, Jered
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Add "clear" to about://instant. Allow users to clear all the log messages in about://instant which makes debugging a lot easier. Remove the old "Reset" button since it becomes confusing which one does what (and that button isn't really that useful anyway). TESTED=navigate to about://instant, click "Clear" BUG=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186270

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : Rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -18 lines) Patch
M chrome/browser/instant/instant_controller.h View 1 2 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/instant/instant_controller.cc View 1 2 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/resources/instant/instant.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/instant/instant.js View 1 2 3 chunks +9 lines, -17 lines 0 comments Download
M chrome/browser/ui/webui/instant_ui.cc View 1 3 chunks +18 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
samarth
This has been bothering me a lot whenever I use about://instant to debug. Please take ...
7 years, 9 months ago (2013-03-04 23:17:58 UTC) #1
sreeram
lgtm
7 years, 9 months ago (2013-03-04 23:25:00 UTC) #2
Evan Stade
lgtm
7 years, 9 months ago (2013-03-05 18:15:21 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/samarth@chromium.org/12377095/6001
7 years, 9 months ago (2013-03-05 18:21:06 UTC) #4
commit-bot: I haz the power
7 years, 9 months ago (2013-03-05 22:28:11 UTC) #5
Message was sent while issue was closed.
Change committed as 186270

Powered by Google App Engine
This is Rietveld 408576698