Remove set_owned_by_client() from one test

In this browser test, the test case owns the view created and the view
is never added to the view hierarchy, so there is no need to call
set_owned_by_client() as the ownership is never transferred.

BUG=1044687

Change-Id: Id5706ab6feaa1d0c22612f575bed93f9529b3d91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2194794Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#767853}

chrome/browser/ui/views/toolbar/browser_actions_container_browsertest.cc

@@ -535,11 +535,10 @@ void BrowserActionsContainerOverflowTest::SetUpOnMainThread() {
   main_bar_ = BrowserView::GetBrowserViewForBrowser(browser())
                   ->toolbar()->browser_actions();
   overflow_parent_ = std::make_unique<views::ResizeAwareParentView>();
-  overflow_parent_->set_owned_by_client();
-  overflow_bar_ = new BrowserActionsContainer(
-      browser(), main_bar_,
-      BrowserView::GetBrowserViewForBrowser(browser())->toolbar(), true);
-  overflow_parent_->AddChildView(overflow_bar_);
+  overflow_bar_ =
+      overflow_parent_->AddChildView(std::make_unique<BrowserActionsContainer>(
+          browser(), main_bar_,
+          BrowserView::GetBrowserViewForBrowser(browser())->toolbar(), true));
 }
 
 void BrowserActionsContainerOverflowTest::TearDownOnMainThread() {