Commit 838630d2 authored by stip's avatar stip Committed by Commit bot
Browse files

Also use EXPECT_NEAR in AllFramesGetDefaultZoom and SubframeRetainsZoomOnNavigation.

BUG=622858

Review-Url: https://codereview.chromium.org/2100233002
Cr-Commit-Position: refs/heads/master@{#402358}
parent 105b4151
......@@ -316,9 +316,13 @@ IN_PROC_BROWSER_TEST_F(IFrameZoomBrowserTest, AllFramesGetDefaultZoom) {
WaitAndCheckFrameZoom(msg_queue, frame_observers);
}
EXPECT_DOUBLE_EQ(
// Make this comparison approximate for Nexus5X test;
// https://crbug.com/622858.
EXPECT_NEAR(
new_default_zoom_factor,
GetMainFrameZoomFactor(web_contents(), main_frame_window_border));
GetMainFrameZoomFactor(web_contents(), main_frame_window_border),
0.01
);
}
IN_PROC_BROWSER_TEST_F(IFrameZoomBrowserTest, SiblingFramesZoom) {
......@@ -415,9 +419,13 @@ IN_PROC_BROWSER_TEST_F(IFrameZoomBrowserTest, SubframeRetainsZoomOnNavigation) {
WaitAndCheckFrameZoom(msg_queue, frame_observers);
}
EXPECT_DOUBLE_EQ(
// Make this comparison approximate for Nexus5X test;
// https://crbug.com/622858.
EXPECT_NEAR(
new_zoom_factor,
GetMainFrameZoomFactor(web_contents(), main_frame_window_border));
GetMainFrameZoomFactor(web_contents(), main_frame_window_border),
0.01
);
// Navigate child frame cross site, and make sure zoom is the same.
TestNavigationObserver observer(web_contents());
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment