Fix "unused variable" warnings.

mash/webtest/webtest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mash/webtest/webtest.h"
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string16.h"
@@ skipping 27 matching lines @@
 class AuraInit;
 }
 
 namespace mash {
 namespace webtest {
 
 class UI : public views::WidgetDelegateView,
            public navigation::mojom::ViewClient {
  public:
   UI(Webtest* webtest,
-     bool is_popup,
      navigation::mojom::ViewPtr view,
      navigation::mojom::ViewClientRequest request)
       : webtest_(webtest),
-        is_popup_(is_popup),
         view_(std::move(view)),
         view_client_binding_(this, std::move(request)) {}
   ~UI() override {
     webtest_->RemoveWindow(GetWidget());
   }
 
   void NavigateTo(const GURL& url) {
     view_->NavigateTo(url);
   }
 
@@ skipping 47 matching lines @@
     current_title_ = base::UTF8ToUTF16(title.get());
     GetWidget()->UpdateWindowTitle();
   }
   void LoadProgressChanged(double progress) override {}
   void ViewCreated(navigation::mojom::ViewPtr view,
                    navigation::mojom::ViewClientRequest request,
                    bool is_popup,
                    mojo::RectPtr initial_rect,
                    bool user_gesture) override {
     views::Widget* window = views::Widget::CreateWindowWithContextAndBounds(
-        new UI(webtest_, is_popup, std::move(view), std::move(request)),
-               nullptr, initial_rect.To<gfx::Rect>());
+        new UI(webtest_, std::move(view), std::move(request)), nullptr,
+        initial_rect.To<gfx::Rect>());
     window->Show();
     webtest_->AddWindow(window);
   }
   void Close() override {
     GetWidget()->Close();
   }
 
   Webtest* webtest_;
-  bool is_popup_;
   mus::Window* content_area_ = nullptr;
   navigation::mojom::ViewPtr view_;
   mojo::Binding<navigation::mojom::ViewClient> view_client_binding_;
   base::string16 current_title_;
 
   DISALLOW_COPY_AND_ASSIGN(UI);
 };
 
 Webtest::Webtest() {}
 Webtest::~Webtest() {}
@@ skipping 33 matching lines @@
     return;
   }
 
   navigation::mojom::ViewFactoryPtr view_factory;
   connector_->ConnectToInterface("exe:navigation", &view_factory);
   navigation::mojom::ViewPtr view;
   navigation::mojom::ViewClientPtr view_client;
   navigation::mojom::ViewClientRequest view_client_request =
       GetProxy(&view_client);
   view_factory->CreateView(std::move(view_client), GetProxy(&view));
-  UI* ui = new UI(this, false, std::move(view), std::move(view_client_request));
+  UI* ui = new UI(this, std::move(view), std::move(view_client_request));
   views::Widget* window = views::Widget::CreateWindowWithContextAndBounds(
       ui, nullptr, gfx::Rect(50, 10, 600, 600));
   ui->NavigateTo(GURL("http://www.theverge.com/"));
   window->Show();
   AddWindow(window);
 }
 
 void Webtest::Create(shell::Connection* connection,
                      mojom::LaunchableRequest request) {
   bindings_.AddBinding(this, std::move(request));
 }
 
 }  // namespace webtest
 }  // namespace mash