2010-12-03 Tony Chang <tony@chromium.org>
authortony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 6 Dec 2010 17:35:05 +0000 (17:35 +0000)
committertony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 6 Dec 2010 17:35:05 +0000 (17:35 +0000)
commit234d0f0a30d9a3b5af208ae238014969f401a521
treec52fe9c4e34aee8777d5bb4c09047c9596db5066
parent88d14caab1562d1025224a433189ef22203cf668
2010-12-03  Tony Chang  <tony@chromium.org>

        Reviewed by Kent Tamura.

        [chromium] fix 2 bugs with inspector tests in DRT
        https://bugs.webkit.org/show_bug.cgi?id=50492

        Tasks can outlive the tasklist (even when canceled) so this was
        causing a crash when ~WebTask() ran.  Avoid this by unregistering
        when a task is canceled.

        Also fix an assert when closing devtool windows by copying some
        logic from test_shell.

        * DumpRenderTree/chromium/Task.cpp:
        (WebTask::~WebTask):
        (TaskList::revokeAll):
        * DumpRenderTree/chromium/Task.h: Canceling a task now removes it from
            the tasklist (since the task can outlive the tasklist).
        * DumpRenderTree/chromium/TestShell.h:
        (TestShell::devToolsWebView):
        * DumpRenderTree/chromium/WebViewHost.cpp:
        (WebViewHost::~WebViewHost): Don't load about:blank when closing
            a window if the window has devtools loaded.  This avoids an
            ASSERT and matches test_shell.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@73373 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/chromium/Task.cpp
WebKitTools/DumpRenderTree/chromium/Task.h
WebKitTools/DumpRenderTree/chromium/TestShell.h
WebKitTools/DumpRenderTree/chromium/WebViewHost.cpp