Skip to content
Snippets Groups Projects
Commit c840d52c authored by rickcam@chromium.org's avatar rickcam@chromium.org
Browse files

Cleaning up some issues discovered in a semi-related code review.

BUG=67206
TEST=manual

Review URL: http://codereview.chromium.org/5858004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69778 0039d316-1c4b-4281-b951-d872f2087c98
parent 1b024c5e
No related merge requests found
......@@ -277,16 +277,15 @@ void BackgroundModeManager::UpdateStatusTrayIconContextMenu() {
menu->AddItem(IDC_OPTIONS, GetPreferencesMenuLabel());
menu->AddItemWithStringId(IDC_TASK_MANAGER, IDS_TASK_MANAGER);
menu->AddSeparator();
int application_position = 0;
int position = 0;
context_menu_application_offset_ = menu->GetItemCount();
for (ExtensionList::const_iterator cursor = applications_.begin();
cursor != applications_.end();
++cursor, ++application_position) {
++cursor, ++position) {
const SkBitmap* icon = applications_.GetIcon(*cursor);
int sort_position = applications_.GetPosition(*cursor);
DCHECK(sort_position == application_position);
DCHECK(position == applications_.GetPosition(*cursor));
const std::string& name = (*cursor)->name();
menu->AddItem(sort_position, ASCIIToUTF16(name));
menu->AddItem(position, UTF8ToUTF16(name));
if (icon)
menu->SetIcon(menu->GetItemCount() - 1, *icon);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment