Add missing ConstrainedDialog code and re-enable disabled PrintPreviewHandlerTest tests on CrOS.

chrome/browser/ui/webui/print_preview_handler_unittest.cc

 // Copyright (c) 2011 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 "base/command_line.h"
 #include "base/json/json_writer.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/printing/background_printing_manager.h"
 #include "chrome/browser/printing/print_preview_tab_controller.h"
@@ skipping 131 matching lines @@
  private:
   void ClearStickySettings() {
     PrintPreviewHandler::last_used_margins_type_ = printing::DEFAULT_MARGINS;
     delete PrintPreviewHandler::last_used_page_size_margins_;
     PrintPreviewHandler::last_used_page_size_margins_ = NULL;
   }
 };
 
 // Test crashs on TouchUI due to initiator tab's native view having no parent.
 // http://crbug.com/104284
-// Test crashes on CrOS as well. http://crbug.com/104403
-#if defined(TOUCH_UI) || defined(OS_CHROMEOS)
+#if defined(TOUCH_UI)
 #define MAYBE_StickyMarginsCustom DISABLED_StickyMarginsCustom
 #else
 #define MAYBE_StickyMarginsCustom StickyMarginsCustom
 #endif
 // Tests that margin settings are saved correctly when printing with custom
 // margins selected.
 TEST_F(PrintPreviewHandlerTest, MAYBE_StickyMarginsCustom) {
   const double kMarginTop = 25.5;
   const double kMarginRight = 26.5;
   const double kMarginBottom = 27.5;
   const double kMarginLeft = 28.5;
   RequestPrintWithCustomMargins(
       kMarginTop, kMarginRight, kMarginBottom, kMarginLeft);
   EXPECT_EQ(1, browser()->tab_count());
 
   // Checking that sticky settings were saved correctly.
   EXPECT_EQ(PrintPreviewHandler::last_used_color_model_, printing::COLOR);
   EXPECT_EQ(PrintPreviewHandler::last_used_margins_type_,
             printing::CUSTOM_MARGINS);
   ASSERT_TRUE(PrintPreviewHandler::last_used_page_size_margins_);
   CheckCustomMargins(kMarginTop, kMarginRight, kMarginBottom, kMarginLeft);
 }
 
-// http://crbug.com/104284, http://crbug.com/104403
-#if defined(TOUCH_UI) || defined(OS_CHROMEOS)
+// http://crbug.com/104284
+#if defined(TOUCH_UI)
 #define MAYBE_StickyMarginsDefault DISABLED_StickyMarginsDefault
 #else
 #define MAYBE_StickyMarginsDefault StickyMarginsDefault
 #endif
 // Tests that margin settings are saved correctly when printing with default
 // margins selected.
 TEST_F(PrintPreviewHandlerTest, MAYBE_StickyMarginsDefault) {
   RequestPrintWithDefaultMargins();
   EXPECT_EQ(1, browser()->tab_count());
 
   // Checking that sticky settings were saved correctly.
   EXPECT_EQ(PrintPreviewHandler::last_used_color_model_, printing::COLOR);
   EXPECT_EQ(PrintPreviewHandler::last_used_margins_type_,
             printing::DEFAULT_MARGINS);
   ASSERT_FALSE(PrintPreviewHandler::last_used_page_size_margins_);
 }
 
-// http://crbug.com/104284, http://crbug.com/104403
-#if defined(TOUCH_UI) || defined(OS_CHROMEOS)
+// http://crbug.com/104284
+#if defined(TOUCH_UI)
 #define MAYBE_StickyMarginsCustomThenDefault \
     DISABLED_StickyMarginsCustomThenDefault
 #else
 #define MAYBE_StickyMarginsCustomThenDefault StickyMarginsCustomThenDefault
 #endif
 // Tests that margin settings are saved correctly when printing with custom
 // margins selected and then again with default margins selected.
 TEST_F(PrintPreviewHandlerTest, MAYBE_StickyMarginsCustomThenDefault) {
   const double kMarginTop = 125.5;
   const double kMarginRight = 126.5;
@@ skipping 12 matching lines @@
   RequestPrintWithDefaultMargins();
 
   // Checking that sticky settings were saved correctly.
   EXPECT_EQ(PrintPreviewHandler::last_used_color_model_, printing::COLOR);
   EXPECT_EQ(PrintPreviewHandler::last_used_margins_type_,
             printing::DEFAULT_MARGINS);
   ASSERT_TRUE(PrintPreviewHandler::last_used_page_size_margins_);
   CheckCustomMargins(kMarginTop, kMarginRight, kMarginBottom, kMarginLeft);
 }
 
-// http://crbug.com/104284, http://crbug.com/104403
-#if defined(TOUCH_UI) || defined(OS_CHROMEOS)
+// http://crbug.com/104284
+#if defined(TOUCH_UI)
 #define MAYBE_GetLastUsedMarginSettingsCustom \
     DISABLED_GetLastUsedMarginSettingsCustom
 #else
 #define MAYBE_GetLastUsedMarginSettingsCustom GetLastUsedMarginSettingsCustom
 #endif
 // Tests that margin settings are retrieved correctly after printing with custom
 // margins.
 TEST_F(PrintPreviewHandlerTest, MAYBE_GetLastUsedMarginSettingsCustom) {
   const double kMarginTop = 125.5;
   const double kMarginRight = 126.5;
@@ skipping 15 matching lines @@
                                          &margin_value));
   EXPECT_EQ(kMarginRight, margin_value);
   EXPECT_TRUE(initial_settings.GetDouble(printing::kSettingMarginBottom,
                                          &margin_value));
   EXPECT_EQ(kMarginBottom, margin_value);
   EXPECT_TRUE(initial_settings.GetDouble(printing::kSettingMarginLeft,
                                          &margin_value));
   EXPECT_EQ(kMarginLeft, margin_value);
 }
 
-// http://crbug.com/104284, http://crbug.com/104403
-#if defined(TOUCH_UI) || defined(OS_CHROMEOS)
+// http://crbug.com/104284
+#if defined(TOUCH_UI)
 #define MAYBE_GetLastUsedMarginSettingsDefault \
     DISABLED_GetLastUsedMarginSettingsDefault
 #else
 #define MAYBE_GetLastUsedMarginSettingsDefault GetLastUsedMarginSettingsDefault
 #endif
 // Tests that margin settings are retrieved correctly after printing with
 // default margins.
 TEST_F(PrintPreviewHandlerTest, MAYBE_GetLastUsedMarginSettingsDefault) {
   RequestPrintWithDefaultMargins();
   base::DictionaryValue initial_settings;
   preview_ui_->handler_->GetLastUsedMarginSettings(&initial_settings);
   int margins_type;
   EXPECT_TRUE(initial_settings.GetInteger(printing::kSettingMarginsType,
                                           &margins_type));
   EXPECT_EQ(margins_type, printing::DEFAULT_MARGINS);
   double margin_value;
   EXPECT_FALSE(initial_settings.GetDouble(printing::kSettingMarginTop,
                                           &margin_value));
   EXPECT_FALSE(initial_settings.GetDouble(printing::kSettingMarginRight,
                                           &margin_value));
   EXPECT_FALSE(initial_settings.GetDouble(printing::kSettingMarginBottom,
                                           &margin_value));
   EXPECT_FALSE(initial_settings.GetDouble(printing::kSettingMarginLeft,
                                           &margin_value));
 }