Allow platform apps to specify a maximum size for the shell container.

chrome/common/extensions/extension.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 "chrome/common/extensions/extension.h"
 
 #include <algorithm>
 
 #include "base/base64.h"
 #include "base/basictypes.h"
@@ skipping 1232 matching lines @@
                            &launch_min_width_,
                            can_specify_size_range,
                            error))
       return false;
   if (!ReadLaunchDimension(manifest_,
                            keys::kLaunchMinHeight,
                            &launch_min_height_,
                            can_specify_size_range,
                            error))
       return false;
+  if (!ReadLaunchDimension(manifest_,
+                           keys::kLaunchMaxWidth,
+                           &launch_max_width_,
+                           can_specify_size_range,
+                           error))
+      return false;
+  if (!ReadLaunchDimension(manifest_,
+                           keys::kLaunchMaxHeight,
+                           &launch_max_height_,
+                           can_specify_size_range,
+                           error))
+      return false;
 
   if (launch_container() == extension_misc::LAUNCH_SHELL) {
     if (!manifest_->Get(keys::kLaunchWidth, &temp)) {
       *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
           errors::kInvalidLaunchValue,
           keys::kLaunchWidth);
       return false;
     }
     if (!manifest_->Get(keys::kLaunchHeight, &temp)) {
       *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
           errors::kInvalidLaunchValue,
           keys::kLaunchHeight);
       return false;
     }
+    if (launch_max_width_ > 0 && launch_max_width_ < launch_min_width_) {
+      *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+          errors::kInvalidLaunchValue,
+          keys::kLaunchMaxWidth);
+      return false;
+    }
+    if (launch_max_height_ > 0 && launch_max_height_ < launch_min_height_) {
+      *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+          errors::kInvalidLaunchValue,
+          keys::kLaunchMaxHeight);
+      return false;
+    }
   }
 
   return true;
 }
 
 bool Extension::LoadAppIsolation(string16* error) {
   Value* temp = NULL;
   if (!manifest_->Get(keys::kIsolation, &temp))
     return true;
 
@@ skipping 227 matching lines @@
       offline_enabled_(false),
       converted_from_user_script_(false),
       background_page_persists_(true),
       manifest_(manifest.release()),
       is_storage_isolated_(false),
       launch_container_(extension_misc::LAUNCH_TAB),
       launch_width_(0),
       launch_height_(0),
       launch_min_width_(0),
       launch_min_height_(0),
+      launch_max_width_(0),
+      launch_max_height_(0),
       wants_file_access_(false),
       creation_flags_(0) {
   DCHECK(path.empty() || path.IsAbsolute());
   path_ = MaybeNormalizePath(path);
 }
 
 Extension::~Extension() {
   if (manifest_)
     delete manifest_;
 }
@@ skipping 1775 matching lines @@
     already_disabled(false),
     extension(extension) {}
 
 UpdatedExtensionPermissionsInfo::UpdatedExtensionPermissionsInfo(
     const Extension* extension,
     const ExtensionPermissionSet* permissions,
     Reason reason)
     : reason(reason),
       extension(extension),
       permissions(permissions) {}