Merge 64807 & 64824 - Cloud Print Dialog work

Merge 64807 - Minimum UI change for Cloud Print dialog.

Minimum change to remove the title and get the initial dialog size correct.
Still has gobs of whitespace at the top.

Merge 64824 - Cloud Print Dialog work (extends 64807)

Allow an HTML dialog to suppress the title bar in a Chromium OS bubble window.
For the Cloud Print dialog, remember the last size requested by the contents.

Bug Url: http://code.google.com/p/chromium-os/issues/detail?id=8494

BUG=chromium-os:8494
TEST=none

Review URL for 64807: http://codereview.chromium.org/4146015
Review URL for 64824: http://codereview.chromium.org/4110010

TBR=scottbyer@google.com

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64977

[Scott Byer, 2010-11-03 19:18:58]

CrOS targeted change, no effect for other platforms.  Combined dependent merges
and resolved conflicts in local branch, compiled and ran tests for CrOS build
and Linux build, original CLs were clean on the trunk.

[Scott Byer, 2010-11-03 21:52:38]

Do you want to double-check the merge patch, or should I submit it TBR (or
you can let me know who else should take a look)?  I've double-checked
myself that it contains what it's supposed to.

-Scott

On Wed, Nov 3, 2010 at 12:18 PM, <scottbyer@chromium.org> wrote:

> Reviewers: zel,
>
> Message:
> CrOS targeted change, no effect for other platforms.  Combined dependent
> merges
> and resolved conflicts in local branch, compiled and ran tests for CrOS
> build
> and Linux build, original CLs were clean on the trunk.
>
> Description:
> Merge 64807 - Minimum UI change for Cloud Print dialog.
>
> Minimum change to remove the title and get the initial dialog size correct.
> Still has gobs of whitespace at the top.
>
> Merge 64824 - Cloud Print Dialog work (extends 64807)
>
> Allow an HTML dialog to suppress the title bar in a Chromium OS bubble
> window.
> For the Cloud Print dialog, remember the last size requested by the
> contents.
>
> Bug Url: http://code.google.com/p/chromium-os/issues/detail?id=8494
>
> BUG=chromium-os:8494
> TEST=none
>
> Review URL for 64807: http://codereview.chromium.org/4146015
> Review URL for 64824: http://codereview.chromium.org/4110010
>
> TBR=scottbyer@google.com
>
> Please review this at http://codereview.chromium.org/4396001/show
>
> SVN Base: svn://svn.chromium.org/chrome/branches/552/src/
>
> Affected files:
>  M     chrome/app/generated_resources.grd
>  M     chrome/browser/browser_signin.cc
>  M     chrome/browser/chrome_plugin_host.cc
>  M     chrome/browser/chromeos/frame/bubble_frame_view.cc
>  M     chrome/browser/chromeos/login/login_html_dialog.h
>  M     chrome/browser/cocoa/html_dialog_window_controller.mm
>  M     chrome/browser/cocoa/html_dialog_window_controller_unittest.mm
>  M     chrome/browser/dom_ui/html_dialog_ui.h
>  M     chrome/browser/gtk/html_dialog_gtk.h
>  M     chrome/browser/modal_html_dialog_delegate.h
>  M     chrome/browser/printing/cloud_print/cloud_print_setup_flow.h
>  M     chrome/browser/printing/print_dialog_cloud.cc
>  M     chrome/browser/printing/print_dialog_cloud_internal.h
>  M     chrome/browser/printing/print_dialog_cloud_unittest.cc
>  M     chrome/browser/remoting/remoting_setup_flow.h
>  M     chrome/browser/remoting/remoting_setup_flow.cc
>  M     chrome/browser/sync/sync_setup_flow.h
>  M     chrome/browser/views/html_dialog_view.h
>  M     chrome/browser/views/html_dialog_view.cc
>  M     chrome/browser/views/html_dialog_view_browsertest.cc
>  M     chrome/browser/views/select_file_dialog.cc
>  M     chrome/common/pref_names.h
>  M     chrome/common/pref_names.cc
>
>
>

[zel, 2010-11-03 21:58:05]

LGTM