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

Side by Side Diff: chrome/browser/ui/views/toolbar/test_toolbar_actions_bar_helper_views.cc

Issue 1104813005: [Extensions Toolbar] Refactor a few test classes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/ui/toolbar/test_toolbar_actions_bar_helper.h"
6
7 #include "chrome/browser/ui/views/toolbar/browser_actions_container.h"
8 #include "ui/views/view.h"
9
10 namespace {
11
12 // The views-specific implementation of the TestToolbarActionsBarHelper, which
13 // creates and owns a BrowserActionsContainer.
14 class TestToolbarActionsBarHelperViews : public TestToolbarActionsBarHelper {
15 public:
16 TestToolbarActionsBarHelperViews(Browser* browser,
17 TestToolbarActionsBarHelperViews* main_bar);
18 ~TestToolbarActionsBarHelperViews() override;
19
20 private:
21 // TestToolbarActionsBarHelper:
22 ToolbarActionsBar* GetToolbarActionsBar() override;
23
24 // The parent of the BrowserActionsContainer, which directly owns the
25 // container as part of the views hierarchy.
26 views::View container_parent_;
27
28 // The created BrowserActionsContainer. Owned by |container_parent_|.
29 BrowserActionsContainer* browser_actions_container_;
30
31 DISALLOW_COPY_AND_ASSIGN(TestToolbarActionsBarHelperViews);
32 };
33
34 TestToolbarActionsBarHelperViews::TestToolbarActionsBarHelperViews(
35 Browser* browser,
36 TestToolbarActionsBarHelperViews* main_bar)
37 : browser_actions_container_(
38 new BrowserActionsContainer(
39 browser,
40 main_bar ? main_bar->browser_actions_container_ : nullptr)) {
41 // The BrowserActionsContainer expects to have a parent (and be added to the
42 // view hierarchy), so wrap it in a shell view.
43 container_parent_.set_owned_by_client();
44 container_parent_.AddChildView(browser_actions_container_);
45 }
46
47 TestToolbarActionsBarHelperViews::~TestToolbarActionsBarHelperViews() {}
48
49 ToolbarActionsBar* TestToolbarActionsBarHelperViews::GetToolbarActionsBar() {
50 return browser_actions_container_->toolbar_actions_bar();
51 }
52
53 } // namespace
54
55 // static
56 scoped_ptr<TestToolbarActionsBarHelper>
57 TestToolbarActionsBarHelper::Create(Browser* browser,
58 TestToolbarActionsBarHelper* main_bar) {
59 return make_scoped_ptr(new TestToolbarActionsBarHelperViews(
60 browser,
61 static_cast<TestToolbarActionsBarHelperViews*>(main_bar)));
62 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/toolbar/browser_action_test_util_views.cc ('k') | chrome/chrome_tests_unit.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698