[Extensions] Code Cleanup - Remove redundant smart-ptr get()s

extensions/browser/api/app_window/app_window_api.cc

 // Copyright (c) 2012 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 "extensions/browser/api/app_window/app_window_api.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/command_line.h"
@@ skipping 169 matching lines @@
                                            create_params.window_key);
         if (existing_window) {
           content::RenderFrameHost* existing_frame =
               existing_window->web_contents()->GetMainFrame();
           int frame_id = MSG_ROUTING_NONE;
           if (render_frame_host()->GetProcess()->GetID() ==
               existing_frame->GetProcess()->GetID()) {
             frame_id = existing_frame->GetRoutingID();
           }
 
-          if (!options->hidden.get() || !*options->hidden.get()) {
-            if (options->focused.get() && !*options->focused.get())
+          if (!options->hidden.get() || !*options->hidden) {
+            if (options->focused.get() && !*options->focused)
               existing_window->Show(AppWindow::SHOW_INACTIVE);
             else
               existing_window->Show(AppWindow::SHOW_ACTIVE);
           }
 
           std::unique_ptr<base::DictionaryValue> result(
               new base::DictionaryValue);
           result->Set("frameId", new base::FundamentalValue(frame_id));
           existing_window->GetSerializedState(result.get());
           result->SetBoolean("existingWindow", true);
@@ skipping 88 matching lines @@
       }
 #if defined(USE_AURA)
       create_params.alpha_enabled = *options->alpha_enabled;
 #else
       // Transparency is only supported on Aura.
       // Fallback to creating an opaque window (by ignoring alphaEnabled).
 #endif
     }
 
     if (options->hidden.get())
-      create_params.hidden = *options->hidden.get();
+      create_params.hidden = *options->hidden;
 
     if (options->resizable.get())
-      create_params.resizable = *options->resizable.get();
+      create_params.resizable = *options->resizable;
 
     if (options->always_on_top.get()) {
-      create_params.always_on_top = *options->always_on_top.get();
+      create_params.always_on_top = *options->always_on_top;
 
       if (create_params.always_on_top &&
           !extension()->permissions_data()->HasAPIPermission(
               APIPermission::kAlwaysOnTopWindows)) {
         error_ = app_window_constants::kAlwaysOnTopPermission;
         return false;
       }
     }
 
     if (options->focused.get())
-      create_params.focused = *options->focused.get();
+      create_params.focused = *options->focused;
 
     if (options->visible_on_all_workspaces.get()) {
       create_params.visible_on_all_workspaces =
-          *options->visible_on_all_workspaces.get();
+          *options->visible_on_all_workspaces;
     }
 
     if (options->type != app_window::WINDOW_TYPE_PANEL) {
       switch (options->state) {
         case app_window::STATE_NONE:
         case app_window::STATE_NORMAL:
           break;
         case app_window::STATE_FULLSCREEN:
           create_params.state = ui::SHOW_STATE_FULLSCREEN;
           break;
@@ skipping 116 matching lines @@
 
     CopyBoundsSpec(inner_bounds, &(params->content_spec));
     CopyBoundsSpec(outer_bounds, &(params->window_spec));
   } else {
     // Parse deprecated fields.
     // Due to a bug in NativeAppWindow::GetFrameInsets() on Windows and ChromeOS
     // the bounds set the position of the window and the size of the content.
     // This will be preserved as apps may be relying on this behavior.
 
     if (options.default_width.get())
-      params->content_spec.bounds.set_width(*options.default_width.get());
+      params->content_spec.bounds.set_width(*options.default_width);
     if (options.default_height.get())
-      params->content_spec.bounds.set_height(*options.default_height.get());
+      params->content_spec.bounds.set_height(*options.default_height);
     if (options.default_left.get())
-      params->window_spec.bounds.set_x(*options.default_left.get());
+      params->window_spec.bounds.set_x(*options.default_left);
     if (options.default_top.get())
-      params->window_spec.bounds.set_y(*options.default_top.get());
+      params->window_spec.bounds.set_y(*options.default_top);
 
     if (options.width.get())
-      params->content_spec.bounds.set_width(*options.width.get());
+      params->content_spec.bounds.set_width(*options.width);
     if (options.height.get())
-      params->content_spec.bounds.set_height(*options.height.get());
+      params->content_spec.bounds.set_height(*options.height);
     if (options.left.get())
-      params->window_spec.bounds.set_x(*options.left.get());
+      params->window_spec.bounds.set_x(*options.left);
     if (options.top.get())
-      params->window_spec.bounds.set_y(*options.top.get());
+      params->window_spec.bounds.set_y(*options.top);
 
     if (options.bounds.get()) {
       app_window::ContentBounds* bounds = options.bounds.get();
       if (bounds->width.get())
-        params->content_spec.bounds.set_width(*bounds->width.get());
+        params->content_spec.bounds.set_width(*bounds->width);
       if (bounds->height.get())
-        params->content_spec.bounds.set_height(*bounds->height.get());
+        params->content_spec.bounds.set_height(*bounds->height);
       if (bounds->left.get())
-        params->window_spec.bounds.set_x(*bounds->left.get());
+        params->window_spec.bounds.set_x(*bounds->left);
       if (bounds->top.get())
-        params->window_spec.bounds.set_y(*bounds->top.get());
+        params->window_spec.bounds.set_y(*bounds->top);
     }
 
     gfx::Size& minimum_size = params->content_spec.minimum_size;
     if (options.min_width.get())
       minimum_size.set_width(*options.min_width);
     if (options.min_height.get())
       minimum_size.set_height(*options.min_height);
     gfx::Size& maximum_size = params->content_spec.maximum_size;
     if (options.max_width.get())
       maximum_size.set_width(*options.max_width);
@@ skipping 58 matching lines @@
 
   if (options.frame->as_frame_options->inactive_color.get()) {
     error_ = app_window_constants::kInactiveColorWithoutColor;
     return false;
   }
 
   return true;
 }
 
 }  // namespace extensions