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

Side by Side Diff: Source/core/layout/LayoutTableCellTest.cpp

Issue 869323003: Oilpan: move RenderObjects off heap. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: review-induced improvements Created 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « Source/core/frame/FrameView.cpp ('k') | Source/core/layout/LayoutTableCol.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2011 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 27 matching lines...) Expand all
38 { 38 {
39 RenderingTest::SetUp(); 39 RenderingTest::SetUp();
40 m_cell = LayoutTableCell::createAnonymous(&document()); 40 m_cell = LayoutTableCell::createAnonymous(&document());
41 } 41 }
42 42
43 virtual void TearDown() 43 virtual void TearDown()
44 { 44 {
45 m_cell->destroy(); 45 m_cell->destroy();
46 } 46 }
47 47
48 RawPtrWillBePersistent<LayoutTableCell> m_cell; 48 LayoutTableCell* m_cell;
49 }; 49 };
50 50
51 TEST_F(LayoutTableCellDeathTest, CanSetColumn) 51 TEST_F(LayoutTableCellDeathTest, CanSetColumn)
52 { 52 {
53 static const unsigned columnIndex = 10; 53 static const unsigned columnIndex = 10;
54 m_cell->setCol(columnIndex); 54 m_cell->setCol(columnIndex);
55 EXPECT_EQ(columnIndex, m_cell->col()); 55 EXPECT_EQ(columnIndex, m_cell->col());
56 } 56 }
57 57
58 TEST_F(LayoutTableCellDeathTest, CanSetColumnToMaxColumnIndex) 58 TEST_F(LayoutTableCellDeathTest, CanSetColumnToMaxColumnIndex)
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 { 107 {
108 setBodyInnerHTML("<table><td rowspan='8190'></td></table>"); 108 setBodyInnerHTML("<table><td rowspan='8190'></td></table>");
109 109
110 LayoutTableCell* cell = toLayoutTableCell(document().body()->firstChild()->f irstChild()->firstChild()->firstChild()->renderer()); 110 LayoutTableCell* cell = toLayoutTableCell(document().body()->firstChild()->f irstChild()->firstChild()->firstChild()->renderer());
111 ASSERT_EQ(cell->rowSpan(), 8190U); 111 ASSERT_EQ(cell->rowSpan(), 8190U);
112 } 112 }
113 113
114 } 114 }
115 115
116 } // namespace blink 116 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/frame/FrameView.cpp ('k') | Source/core/layout/LayoutTableCol.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698