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

Issue 5858004: Clean-up BackgroundModeManager-BackgroundApplicationListModel interaction (Closed)

Created:
10 years ago by The wrong rickcam account
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Cleaning up some issues discovered in a semi-related code review. BUG=67206 TEST=manual Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=69778

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -5 lines) Patch
M chrome/browser/background_mode_manager.cc View 1 chunk +4 lines, -5 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
Andrew T Wilson (Slow)
http://codereview.chromium.org/5858004/diff/1/chrome/browser/background_mode_manager.cc File chrome/browser/background_mode_manager.cc (right): http://codereview.chromium.org/5858004/diff/1/chrome/browser/background_mode_manager.cc#newcode288 chrome/browser/background_mode_manager.cc:288: menu->AddItem(position, UTF8ToUTF16(name)); LGTM.
10 years ago (2010-12-17 01:56:22 UTC) #1
Andrew T Wilson (Slow)
10 years ago (2010-12-17 01:56:47 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698