LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid.html - Issue 532203002: Test: Don't add scrollbar width to intrinsic width of fixed-width grid items

Unified Diff: LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid.html

Issue 532203002: Test: Don't add scrollbar width to intrinsic width of fixed-width grid items (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Incorporating review comments Created 6 years, 3 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

« no previous file with comments | « no previous file | LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid.html

diff --git a/LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid.html b/LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid.html

new file mode 100644

index 0000000000000000000000000000000000000000..5d36f3baa03319c18c830f503e0a544a276c89ad

--- /dev/null

+++ b/LayoutTests/fast/css-grid-layout/fixed-width-intrinsic-width-should-exclude-scrollbar-width-in-grid.html

@@ -0,0 +1,30 @@

+<!DOCTYPE html>

+<html>

+<link href="resources/grid.css" rel="stylesheet">

+<style>

+.grid {

+ grid-template-columns: 200px 200px;

+ grid-template-rows: 200px 200px;

Julien - ping for review 2014/09/05 17:32:13 I would change the grid cells' sizes to have more

Sunil Ratnu 2014/09/06 12:36:10 Done.

+.gridItemScrollOverflow {

+ width: 50px;

+ height: 50px;

+ overflow: scroll;

+</style>

+<script src="../../resources/check-layout.js"></script>

+<body onload="checkLayout('.grid')">

+<p>This test checks that scrollbar width should not be added to the width of fixed-width grid items.</p>

+<div class="grid">

+ <div class="firstRowFirstColumn gridItemScrollOverflow" data-expected-width="50" data-expected-height="50"></div>

+ <div class="firstRowSecondColumn gridItemScrollOverflow" data-expected-width="50" data-expected-height="50"></div>

+ <div class="secondRowFirstColumn gridItemScrollOverflow" data-expected-width="50" data-expected-height="50"></div>

+ <div class="secondRowSecondColumn gridItemScrollOverflow" data-expected-width="50" data-expected-height="50"></div>

Julien - ping for review 2014/09/05 17:32:13 We will want one of the grid items to be also a gr

Sunil Ratnu 2014/09/06 12:36:10 Done.

+</div>

+</body>

+</html>