My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 45595: ExtensionApiTest.Bookmarks "crashed"
2 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  erik...@chromium.org
Closed:  Oct 2010
Cc:  j...@chromium.org, feldst...@chromium.org

Restricted
  • Only users with Commit permission may comment.


Sign in to add a comment
 
Project Member Reported by jshin@chromium.org, Jun 1, 2010
The test didn't complete (crashed) on XP Tests (dbg) (4) and has the 
following backtrace (apparently after running all that needs to be run)


Note: Google Test filter = ExtensionApiTest.Bookmarks
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExtensionApiTest
[ RUN      ] ExtensionApiTest.Bookmarks
Got EXTENSION_LOADED notification.
( RUN      ) getTree
Got EXTENSION_HOST_DID_STOP_LOADING notification.
(  SUCCESS )
( RUN      ) get
(  SUCCESS )
( RUN      ) getArray
(  SUCCESS )
( RUN      ) getChildren
(  SUCCESS )
( RUN      ) create
(  SUCCESS )
( RUN      ) createFolder
(  SUCCESS )
( RUN      ) moveSetup
(  SUCCESS )
( RUN      ) move
(  SUCCESS )
( RUN      ) search
(  SUCCESS )
( RUN      ) update
(  SUCCESS )
( RUN      ) remove
(  SUCCESS )
( RUN      ) removeTree
(  SUCCESS )
( RUN      ) quotaLimitedCreate
(  SUCCESS )
( RUN      ) quotaSetup
(  SUCCESS )
( RUN      ) quotaLimitedUpdate
(  SUCCESS )
( RUN      ) getRecentSetup
(  SUCCESS )
( RUN      ) getRecent
(  SUCCESS )
Got EXTENSION_TEST_PASSED notification.
Backtrace:
	RevokeDragDrop [0x77532BE5+144]
	(No symbol) [0x02329DE0]
	(No symbol) [0x02308CB9]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x008A150B+3398987]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x008A0E87+3397319]
	(No symbol) [0x0232C3DE]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x0136B67E+14712510]
	GetDC [0x7E418734+109]
	GetDC [0x7E418816+335]
	DefWindowProcW [0x7E428EA0+384]
	DefWindowProcW [0x7E428EEC+460]
	KiUserCallbackDispatcher [0x7C90E473+19]
	(No symbol) [0x02331C0F]
	(No symbol) [0x02331620]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x013ABC69+14976169]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x013ABD15+14976341]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x013AC269+14977705]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x014246C4+15470340]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x01423F32+15468402]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x013AB3FF+14974015]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x013AB1EB+14973483]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x013AC9B9+14979577]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x012FF6D7+14270231]
	(No symbol) [0x0041963C]
	(No symbol) [0x0041932C]
	(No symbol) [0x00430B8C]
	(No symbol) [0x0042BC31]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x0056589F+8415]
	(No symbol) [0x00414E12]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x014F0263+16304803]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x014F0EF3+16308019]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x014F1491+16309457]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x014F66D2+16330514]
	RelaunchChromeBrowserWithNewCommandLineIfNeeded 
[0x014F5336+16325494]
	(No symbol) [0x0043B7DE]
	(No symbol) [0x0043B59F]
	(No symbol) [0x0054824A]
	RegisterWaitForInputIdle [0x7C817077+73]
Jun 1, 2010
#1 jshin@chromium.org
(No comment was entered for this change.)
Labels: OS-Windows
Jun 1, 2010
#2 jshin@chromium.org
I'm disabling it on Windows for now. 


Owner: erik...@chromium.org
Jun 1, 2010
#3 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=48683 

------------------------------------------------------------------------
r48683 | jshin@chromium.org | 2010-06-01 17:24:35 -0700 (Tue, 01 Jun 2010) | 7 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_apitest.cc?r1=48683&r2=48682
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/find_bar_host_browsertest.cc?r1=48683&r2=48682

Disable FindInPageControllerTest.ActivateLinkNavigatesPage and 
ExtensionApiTest.Bookmarks on Windows

BUG=45594,45595
TEST=Windows bot don't turn red with browser_test failure
TBR=jrg
Review URL: http://codereview.chromium.org/2433003
------------------------------------------------------------------------

Jun 14, 2010
#4 rafaelw@chromium.org
(No comment was entered for this change.)
Status: Assigned
Aug 2, 2010
#5 phajdan.jr@chromium.org
(No comment was entered for this change.)
Labels: Tests-Disabled
Oct 11, 2010
#6 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=62170

------------------------------------------------------------------------
r62170 | erikkay@chromium.org | Mon Oct 11 12:24:41 PDT 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_module.cc?r1=62170&r2=62169&pathrev=62170
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/resources/extension_apitest.js?r1=62170&r2=62169&pathrev=62170
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/bookmarks/test.js?r1=62170&r2=62169&pathrev=62170
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_apitest.cc?r1=62170&r2=62169&pathrev=62170

fix bookmarks API crash when handling invalid URL
also re-enable test on Windows (can't repro crashes)

BUG=56945,45595
TEST=ExtensionApiTest.Bookmarks

Review URL: http://codereview.chromium.org/3672003
------------------------------------------------------------------------
Summary: ExtensionApiTest.Bookmarks "crashed"
Oct 11, 2010
#7 erik...@chromium.org
No crashes since I landed it.  Knock on wood.
Status: Fixed
Oct 12, 2012
#8 bugdroid1@chromium.org
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Labels: Restrict-AddIssueComment-Commit
Mar 10, 2013
#9 bugdroid1@chromium.org
(No comment was entered for this change.)
Labels: -Area-UI -Feature-Extensions -Tests-Disabled Cr-Platform-Extensions Cr-UI Cr-Tests-Disabled
Sign in to add a comment

Powered by Google Project Hosting