Make 'options' and callback arg to chrome.appWindow.create optional.

chrome/browser/extensions/api/app_window/app_window_api.cc

Index: chrome/browser/extensions/api/app_window/app_window_api.cc
diff --git a/chrome/browser/extensions/api/app_window/app_window_api.cc b/chrome/browser/extensions/api/app_window/app_window_api.cc
index edb97d6308771f6036289c04e42427a8f3764a06..058a804f2310fbc0a919839b9dd44faedaa1a092 100644
--- a/chrome/browser/extensions/api/app_window/app_window_api.cc
+++ b/chrome/browser/extensions/api/app_window/app_window_api.cc
@@ -31,14 +31,16 @@ bool AppWindowCreateFunction::RunImpl() {
   // ShellWindow::Create so we can set the opener at create time rather than
   // with a hack in AppWindowCustomBindings::GetView().
   ShellWindow::CreateParams create_params;
-  if (params->options.width.get())
-    create_params.bounds.set_width(*params->options.width.get());
-  if (params->options.height.get())
-    create_params.bounds.set_height(*params->options.height.get());
-  if (params->options.left.get())
-    create_params.bounds.set_x(*params->options.left.get());
-  if (params->options.top.get())
-    create_params.bounds.set_y(*params->options.top.get());
+  if (params->options.get()) {

[not at google - send to devlin, 2012/05/28 04:46:43]

nit: save a reference to "params->options.get()" rather than typing it
repeatedly.

[jeremya, 2012/05/28 06:49:01]

Done.

+    if (params->options->width.get())
+      create_params.bounds.set_width(*params->options->width.get());
+    if (params->options->height.get())
+      create_params.bounds.set_height(*params->options->height.get());
+    if (params->options->left.get())
+      create_params.bounds.set_x(*params->options->left.get());
+    if (params->options->top.get())
+      create_params.bounds.set_y(*params->options->top.get());
+  }
   ShellWindow* shell_window =
       ShellWindow::Create(profile(), GetExtension(), url, create_params);
   shell_window->Show();