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

Side by Side Diff: chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.cc

Issue 2684723003: Remove LauncherItemController::Launch, cleanup. (Closed)
Patch Set: Cleanup Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h" 5 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h"
6 6
7 #include "base/strings/string_util.h" 7 #include "base/strings/string_util.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/profiles/profile_manager.h" 10 #include "chrome/browser/profiles/profile_manager.h"
(...skipping 16 matching lines...) Expand all
27 mojo::AssociatedGroup* associated_group) { 27 mojo::AssociatedGroup* associated_group) {
28 DCHECK(!item_delegate_binding_.is_bound()); 28 DCHECK(!item_delegate_binding_.is_bound());
29 ash::mojom::ShelfItemDelegateAssociatedPtrInfo ptr_info; 29 ash::mojom::ShelfItemDelegateAssociatedPtrInfo ptr_info;
30 item_delegate_binding_.Bind(&ptr_info, associated_group); 30 item_delegate_binding_.Bind(&ptr_info, associated_group);
31 return ptr_info; 31 return ptr_info;
32 } 32 }
33 33
34 private: 34 private:
35 // ash::mojom::ShelfItemDelegate: 35 // ash::mojom::ShelfItemDelegate:
36 void LaunchItem() override { 36 void LaunchItem() override {
37 controller_->LaunchApp(app_id_, ash::LAUNCH_FROM_UNKNOWN, ui::EF_NONE); 37 controller_->LaunchApp(app_id_, std::string(), ash::LAUNCH_FROM_UNKNOWN,
38 ui::EF_NONE);
38 } 39 }
39 void ExecuteCommand(uint32_t command_id, int32_t event_flags) override { 40 void ExecuteCommand(uint32_t command_id, int32_t event_flags) override {
40 NOTIMPLEMENTED(); 41 NOTIMPLEMENTED();
41 } 42 }
42 void ItemPinned() override { NOTIMPLEMENTED(); } 43 void ItemPinned() override { NOTIMPLEMENTED(); }
43 void ItemUnpinned() override { NOTIMPLEMENTED(); } 44 void ItemUnpinned() override { NOTIMPLEMENTED(); }
44 void ItemReordered(uint32_t order) override { NOTIMPLEMENTED(); } 45 void ItemReordered(uint32_t order) override { NOTIMPLEMENTED(); }
45 46
46 std::string app_id_; 47 std::string app_id_;
47 mojo::AssociatedBinding<ash::mojom::ShelfItemDelegate> item_delegate_binding_; 48 mojo::AssociatedBinding<ash::mojom::ShelfItemDelegate> item_delegate_binding_;
(...skipping 249 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 app_id_to_item_delegate_.insert( 298 app_id_to_item_delegate_.insert(
298 std::make_pair(app_id, std::move(delegate))); 299 std::make_pair(app_id, std::move(delegate)));
299 300
300 AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id); 301 AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id);
301 if (app_icon_loader) { 302 if (app_icon_loader) {
302 app_icon_loader->FetchImage(app_id); 303 app_icon_loader->FetchImage(app_id);
303 app_icon_loader->UpdateImage(app_id); 304 app_icon_loader->UpdateImage(app_id);
304 } 305 }
305 } 306 }
306 } 307 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698