Re: Issue 113008 in chromium: [Windows] Failed to unregister class Chrome_WidgetWin_0 on shutdown

[chro...@googlecode.com]

Updates:
Labels: -Area-UI Area-Internals Tests

Comment #6 on issue 113008 by bau...@chromium.org: [Windows] Failed to
unregister class Chrome_WidgetWin_0 on shutdown
http://code.google.com/p/chromium/issues/detail?id=113008

This also breaks unit tests that set up a ShadowingAtExitManager (the first
time the ClassRegistrar is destroyed, it fails to unregister the window
class, then it fails to re-register (because it is already registered,
which triggers a CHECK).

[chro...@googlecode.com]

Comment #7 on issue 113008 by bugdro...@chromium.org: [Windows] Failed to

unregister class Chrome_WidgetWin_0 on shutdown

http://code.google.com/p/chromium/issues/detail?id=113008#c7

The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=169938

------------------------------------------------------------------------
r169938 | bau...@chromium.org | 2012-11-28T14:45:46.440931Z

Changed paths:
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_preview_test.cc?r1=169938&r2=169937&pathrev=169938
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_preview_test.h?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/base/browser_with_test_window_test.h?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/ui/base/win/window_impl.cc?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/webui/print_preview/print_preview_ui_unittest.cc?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_tests_unit.gypi?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/ui/base/win/window_impl.h?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_preview_tab_controller_unittest.cc?r1=169938&r2=169937&pathrev=169938
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_preview_tab_controller.cc?r1=169938&r2=169937&pathrev=169938

Reland r167487: Get full WebPluginInfo for the PDF plug-in before enabling
it for print preview.

Previous review: http://codereview.chromium.org/11364202/

TBR=the...@chromium.org
BUG=159902,113008

Review URL: https://chromiumcodereview.appspot.com/11417003
------------------------------------------------------------------------

[chro...@googlecode.com]

Updates:
Labels: merge-merged-1312

Comment #8 on issue 113008 by bugdro...@chromium.org: [Windows] Failed to

unregister class Chrome_WidgetWin_0 on shutdown

http://code.google.com/p/chromium/issues/detail?id=113008#c8

The following revision refers to this bug:

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

------------------------------------------------------------------------
r170690 | bau...@chromium.org | 2012-12-02T18:20:15.900582Z

Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/test/base/browser_with_test_window_test.h?r1=170690&r2=170689&pathrev=170690
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/ui/base/win/window_impl.cc?r1=170690&r2=170689&pathrev=170690
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/browser/ui/webui/print_preview/print_preview_ui_unittest.cc?r1=170690&r2=170689&pathrev=170690
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/ui/base/win/window_impl.h?r1=170690&r2=170689&pathrev=170690
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/browser/printing/print_preview_tab_controller_unittest.cc?r1=170690&r2=170689&pathrev=170690
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/chrome_tests.gypi?r1=170690&r2=170689&pathrev=170690
M
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/browser/printing/print_preview_tab_controller.cc?r1=170690&r2=170689&pathrev=170690
A
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/browser/printing/print_preview_test.cc?r1=170690&r2=170689&pathrev=170690
A
http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/browser/printing/print_preview_test.h?r1=170690&r2=170689&pathrev=170690

Merge r169938: Reland r167487: Get full WebPluginInfo for the PDF plug-in

before enabling it for print preview.

Previous review: http://codereview.chromium.org/11364202/

TBR=the...@chromium.org
BUG=159902,113008

Review URL: https://chromiumcodereview.appspot.com/11417003

Conflicts:

chrome/chrome_tests_unit.gypi

Review URL: https://codereview.chromium.org/11316300
------------------------------------------------------------------------