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

Side by Side Diff: services/window_manager/window_manager_api_unittest.cc

Issue 868463008: Remove Client relationship between mojo.Shell/mojo.Application (Closed) Base URL: git@github.com:domokit/mojo.git@app_impl_init
Patch Set: fix android Created 5 years, 11 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "base/bind.h" 5 #include "base/bind.h"
6 #include "base/command_line.h" 6 #include "base/command_line.h"
7 #include "base/memory/scoped_vector.h" 7 #include "base/memory/scoped_vector.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "mojo/public/cpp/application/application_delegate.h" 9 #include "mojo/public/cpp/application/application_delegate.h"
10 #include "mojo/public/cpp/application/application_impl.h" 10 #include "mojo/public/cpp/application/application_impl.h"
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 typedef base::Callback<void(View*)> RootAddedCallback; 76 typedef base::Callback<void(View*)> RootAddedCallback;
77 77
78 explicit TestApplicationLoader(const RootAddedCallback& root_added_callback) 78 explicit TestApplicationLoader(const RootAddedCallback& root_added_callback)
79 : root_added_callback_(root_added_callback) {} 79 : root_added_callback_(root_added_callback) {}
80 ~TestApplicationLoader() override {} 80 ~TestApplicationLoader() override {}
81 81
82 private: 82 private:
83 // Overridden from mojo::ApplicationLoader: 83 // Overridden from mojo::ApplicationLoader:
84 void Load(mojo::ApplicationManager* application_manager, 84 void Load(mojo::ApplicationManager* application_manager,
85 const GURL& url, 85 const GURL& url,
86 mojo::ShellPtr shell, 86 mojo::InterfaceRequest<mojo::Application> application_request,
87 LoadCallback callback) override { 87 LoadCallback callback) override {
88 ASSERT_TRUE(shell); 88 ASSERT_TRUE(application_request.is_pending());
89 scoped_ptr<ApplicationImpl> app(new ApplicationImpl(this, shell.Pass())); 89 scoped_ptr<ApplicationImpl> app(
90 new ApplicationImpl(this, application_request.Pass()));
90 apps_.push_back(app.release()); 91 apps_.push_back(app.release());
91 } 92 }
92 void OnApplicationError(mojo::ApplicationManager* application_manager, 93 void OnApplicationError(mojo::ApplicationManager* application_manager,
93 const GURL& url) override {} 94 const GURL& url) override {}
94 95
95 // Overridden from mojo::ApplicationDelegate: 96 // Overridden from mojo::ApplicationDelegate:
96 void Initialize(ApplicationImpl* app) override { 97 void Initialize(ApplicationImpl* app) override {
97 view_manager_client_factory_.reset( 98 view_manager_client_factory_.reset(
98 new mojo::ViewManagerClientFactory(app->shell(), this)); 99 new mojo::ViewManagerClientFactory(app->shell(), this));
99 } 100 }
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 258
258 Id second_window = OpenWindow(); 259 Id second_window = OpenWindow();
259 window_manager_->ActivateWindow(second_window, 260 window_manager_->ActivateWindow(second_window,
260 base::Bind(&EmptyResultCallback)); 261 base::Bind(&EmptyResultCallback));
261 ids = WaitForActiveWindowChange(); 262 ids = WaitForActiveWindowChange();
262 EXPECT_EQ(ids.first, first_window); 263 EXPECT_EQ(ids.first, first_window);
263 EXPECT_EQ(ids.second, second_window); 264 EXPECT_EQ(ids.second, second_window);
264 } 265 }
265 266
266 } // namespace window_manager 267 } // namespace window_manager
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698