Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2153)

Unified Diff: content/shell/renderer/layout_test/blink_test_runner.h

Issue 1685003002: Plumb the correct owner document through DocumentInit::m_owner. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/shell/renderer/layout_test/blink_test_runner.h
diff --git a/content/shell/renderer/layout_test/blink_test_runner.h b/content/shell/renderer/layout_test/blink_test_runner.h
index fc5fe51d989e434e8da3070a31bc79f33f72ce7a..b2598fe28fa63cc7553b506c8229bc3fd6d54439 100644
--- a/content/shell/renderer/layout_test/blink_test_runner.h
+++ b/content/shell/renderer/layout_test/blink_test_runner.h
@@ -157,7 +157,12 @@ class BlinkTestRunner : public RenderViewObserver,
blink::WebPoint ConvertDIPToNative(
const blink::WebPoint& point_in_dip) const override;
- void Reset();
+ // Resets a RenderView to a known state for layout tests. It is used both when
+ // a RenderView is created and when reusing an existing RenderView for the
+ // next test case.
+ // When reusing an existing RenderView, |for_new_test| should be true, which
+ // also resets additional state, like the main frame's name and opener.
+ void Reset(bool for_new_test);
void set_proxy(test_runner::WebTestProxyBase* proxy) { proxy_ = proxy; }
test_runner::WebTestProxyBase* proxy() const { return proxy_; }

Powered by Google App Engine
This is Rietveld 408576698