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

Issue 2374523002: Add logging for prefetch tablist task. (Closed)

Created:
4 years, 2 months ago by Peter Wen
Modified:
4 years, 2 months ago
Reviewers:
Theresa, gone
CC:
chromium-reviews, agrieve+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add logging for prefetch tablist task. Also clean-up for unnecessary context parameters. BUG=649729 Committed: https://crrev.com/e9d894c77a305b224e252637c2c2c05f3275c17d Cr-Commit-Position: refs/heads/master@{#421273}

Patch Set 1 #

Patch Set 2 : Add logging. #

Messages

Total messages: 17 (9 generated)
Peter Wen
🚢
4 years, 2 months ago (2016-09-26 20:32:03 UTC) #4
Theresa
lgtm
4 years, 2 months ago (2016-09-26 20:53:49 UTC) #5
Theresa
While you're in here making changes, will you please add a logging statement at the ...
4 years, 2 months ago (2016-09-26 23:42:50 UTC) #8
Peter Wen
Added Log.w so we have more debug information. +dfalcantara for *tabmodel/ OWNERS when he gets ...
4 years, 2 months ago (2016-09-27 13:13:50 UTC) #10
gone
lgtm
4 years, 2 months ago (2016-09-27 17:27:37 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2374523002/20001
4 years, 2 months ago (2016-09-27 17:40:05 UTC) #14
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 2 months ago (2016-09-27 18:22:37 UTC) #15
commit-bot: I haz the power
4 years, 2 months ago (2016-09-27 18:33:53 UTC) #17
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/e9d894c77a305b224e252637c2c2c05f3275c17d
Cr-Commit-Position: refs/heads/master@{#421273}

Powered by Google App Engine
This is Rietveld 408576698