[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc
tkent at chromium.org
tkent at chromium.org
Wed Dec 22 12:31:37 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit c70650e665c56d37c11b242ba5826fd14592d85d
Author: tkent at chromium.org <tkent at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Wed Aug 25 02:59:44 2010 +0000
2010-08-24 Kent Tamura <tkent at chromium.org>
Reviewed by Dimitri Glazkov.
[DRT/Chromium] Fix 28 inspector test crashes
https://bugs.webkit.org/show_bug.cgi?id=44574
* platform/chromium/drt_expectations.txt:
2010-08-24 Kent Tamura <tkent at chromium.org>
Reviewed by Dimitri Glazkov.
[DRT/Chromium] Fix 28 inspector test crashes
https://bugs.webkit.org/show_bug.cgi?id=44574
Reset DevToolsClient and DevToolsAgent before closing a DevTools
WebView in order to avoid accessing to a deleted WebView instance.
* DumpRenderTree/chromium/TestShell.cpp:
(TestShell::closeDevTools):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@65966 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index f0138e6..3361434 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,12 @@
+2010-08-24 Kent Tamura <tkent at chromium.org>
+
+ Reviewed by Dimitri Glazkov.
+
+ [DRT/Chromium] Fix 28 inspector test crashes
+ https://bugs.webkit.org/show_bug.cgi?id=44574
+
+ * platform/chromium/drt_expectations.txt:
+
2010-08-24 Ojan Vafai <ojan at chromium.org>
Reviewed by Darin Adler.
diff --git a/LayoutTests/platform/chromium/drt_expectations.txt b/LayoutTests/platform/chromium/drt_expectations.txt
index 1e78f8c..d9a16ae 100644
--- a/LayoutTests/platform/chromium/drt_expectations.txt
+++ b/LayoutTests/platform/chromium/drt_expectations.txt
@@ -208,29 +208,17 @@ BUG50197 WIN MAC LINUX : svg/dynamic-updates/SVGRadialGradientElement-svgdom-gra
// Need to implement something?
BUG_DRT WIN : http/tests/inspector-enabled/console-log-before-frame-navigation.html = CRASH
-BUG_DRT WIN : http/tests/inspector/change-iframe-src.html = CRASH
BUG_DRT WIN : http/tests/inspector/console-resource-errors.html = CRASH
-BUG_DRT WIN : http/tests/inspector/console-xhr-logging.html = CRASH
BUG_DRT WIN : http/tests/inspector/inspect-iframe-from-different-domain.html = CRASH
-BUG_DRT WIN : http/tests/inspector/resource-har-conversion.html = CRASH
BUG_DRT WIN : http/tests/inspector/resource-parameters.html = CRASH
-BUG_DRT WIN : inspector/audits-panel-functional.html = CRASH
BUG_DRT WIN : inspector/console-assert.html = CRASH
-BUG_DRT WIN : inspector/console-clear.html = CRASH
BUG_DRT WIN : inspector/console-command-clear.html = CRASH
-BUG_DRT WIN : inspector/console-dir-global.html = CRASH
BUG_DRT WIN : inspector/console-dir.html = CRASH
-BUG_DRT WIN : inspector/console-dirxml.html = CRASH
BUG_DRT WIN : inspector/console-eval-global.html = CRASH
-BUG_DRT WIN : inspector/console-eval.html = CRASH
BUG_DRT WIN : inspector/console-format-collections.html = CRASH
-BUG_DRT WIN : inspector/console-format.html = CRASH
BUG_DRT WIN : inspector/console-log-before-inspector-open.html = CRASH
-BUG_DRT WIN : inspector/console-log-syntax-error.html = CRASH
BUG_DRT WIN : inspector/console-tests.html = CRASH
-BUG_DRT WIN : inspector/console-trace.html = CRASH
BUG_DRT WIN : inspector/cookie-resource-match.html = CRASH
-BUG_DRT WIN : inspector/debugger-breakpoints-not-activated-on-reload.html = CRASH
BUG_DRT WIN : inspector/debugger-eval-while-paused.html = CRASH
BUG_DRT WIN : inspector/debugger-no-nested-pause.html = CRASH
BUG_DRT WIN : inspector/debugger-pause-in-eval-script.html = CRASH
@@ -240,37 +228,21 @@ BUG_DRT WIN : inspector/debugger-step-in.html = CRASH
BUG_DRT WIN : inspector/debugger-step-out.html = CRASH
BUG_DRT WIN : inspector/debugger-step-over.html = CRASH
BUG_DRT WIN : inspector/dom-breakpoints.html = CRASH
-BUG_DRT WIN : inspector/elements-img-tooltip.html = CRASH
BUG_DRT WIN : inspector/elements-panel-limited-children.html = CRASH
-BUG_DRT WIN : inspector/elements-panel-rewrite-href.html = CRASH
BUG_DRT WIN : inspector/elements-panel-search.html = CRASH
-BUG_DRT WIN : inspector/elements-panel-selection-on-refresh.html = CRASH
BUG_DRT WIN : inspector/elements-panel-structure.html = CRASH
-BUG_DRT WIN : inspector/elements-panel-styles.html = CRASH
BUG_DRT WIN : inspector/elements-panel-xhtml-structure.xhtml = CRASH
-BUG_DRT WIN : inspector/evaluate-in-frontend.html = CRASH
BUG_DRT WIN : inspector/inspected-objects-not-overriden.html = CRASH
-BUG_DRT WIN : inspector/styles-computed-trace.html = CRASH
BUG_DRT WIN : inspector/styles-disable-inherited.html = CRASH
-BUG_DRT WIN : inspector/styles-disable-then-delete.html = CRASH
BUG_DRT WIN : inspector/styles-disable-then-enable.html = CRASH
-BUG_DRT WIN : inspector/styles-iframe.html = CRASH
BUG_DRT WIN : inspector/styles-source-lines-inline.html = CRASH
-BUG_DRT WIN : inspector/styles-source-lines.html = CRASH
BUG_DRT WIN : inspector/styles-source-offsets.html = CRASH
-BUG_DRT WIN : inspector/syntax-highlight-css.html = CRASH
BUG_DRT WIN : inspector/syntax-highlight-html.html = CRASH
-BUG_DRT WIN : inspector/syntax-highlight-javascript.html = CRASH
BUG_DRT WIN : inspector/timeline-enum-stability.html = CRASH
-BUG_DRT WIN : inspector/timeline-event-dispatch.html = CRASH
BUG_DRT WIN : inspector/timeline-layout.html = CRASH
-BUG_DRT WIN : inspector/timeline-mark-timeline.html = CRASH
BUG_DRT WIN : inspector/timeline-network-resource.html = CRASH
-BUG_DRT WIN : inspector/timeline-paint.html = CRASH
BUG_DRT WIN : inspector/timeline-parse-html.html = CRASH
-BUG_DRT WIN : inspector/timeline-recalculate-styles.html = CRASH
BUG_DRT WIN : inspector/timeline-script-tag-1.html = CRASH
-BUG_DRT WIN : inspector/timeline-script-tag-2.html = CRASH
BUG_DRT WIN : inspector/timeline-trivial.html = CRASH
diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
index 2f07142..16f6050 100644
--- a/WebKitTools/ChangeLog
+++ b/WebKitTools/ChangeLog
@@ -1,3 +1,16 @@
+2010-08-24 Kent Tamura <tkent at chromium.org>
+
+ Reviewed by Dimitri Glazkov.
+
+ [DRT/Chromium] Fix 28 inspector test crashes
+ https://bugs.webkit.org/show_bug.cgi?id=44574
+
+ Reset DevToolsClient and DevToolsAgent before closing a DevTools
+ WebView in order to avoid accessing to a deleted WebView instance.
+
+ * DumpRenderTree/chromium/TestShell.cpp:
+ (TestShell::closeDevTools):
+
2010-08-24 Dirk Pranke <dpranke at chromium.org>
Reviewed by Ojan Vafai.
diff --git a/WebKitTools/DumpRenderTree/chromium/TestShell.cpp b/WebKitTools/DumpRenderTree/chromium/TestShell.cpp
index f5b0ced..0e9c56a 100644
--- a/WebKitTools/DumpRenderTree/chromium/TestShell.cpp
+++ b/WebKitTools/DumpRenderTree/chromium/TestShell.cpp
@@ -144,6 +144,10 @@ void TestShell::showDevTools()
void TestShell::closeDevTools()
{
if (m_devTools) {
+ m_drtDevToolsAgent->reset();
+ if (m_drtDevToolsClient)
+ m_drtDevToolsClient->reset();
+ m_drtDevToolsAgent->setWebView(0);
closeWindow(m_devTools);
m_devTools = 0;
}
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list