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

Side by Side Diff: Source/core/rendering/RenderView.h

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/rendering/RenderVTTCue.cpp ('k') | Source/core/rendering/RenderView.cpp » ('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) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * Copyright (C) 2006 Apple Computer, Inc. 3 * Copyright (C) 2006 Apple Computer, Inc.
4 * 4 *
5 * This library is free software; you can redistribute it and/or 5 * This library is free software; you can redistribute it and/or
6 * modify it under the terms of the GNU Library General Public 6 * modify it under the terms of the GNU Library General Public
7 * License as published by the Free Software Foundation; either 7 * License as published by the Free Software Foundation; either
8 * version 2 of the License, or (at your option) any later version. 8 * version 2 of the License, or (at your option) any later version.
9 * 9 *
10 * This library is distributed in the hope that it will be useful, 10 * This library is distributed in the hope that it will be useful,
(...skipping 27 matching lines...) Expand all
38 class RenderQuote; 38 class RenderQuote;
39 39
40 // The root of the render tree, corresponding to the CSS initial containing bloc k. 40 // The root of the render tree, corresponding to the CSS initial containing bloc k.
41 // It's dimensions match that of the logical viewport (which may be different fr om 41 // It's dimensions match that of the logical viewport (which may be different fr om
42 // the visible viewport in fixed-layout mode), and it is always at position (0,0 ) 42 // the visible viewport in fixed-layout mode), and it is always at position (0,0 )
43 // relative to the document (and so isn't necessarily in view). 43 // relative to the document (and so isn't necessarily in view).
44 class RenderView final : public RenderBlockFlow { 44 class RenderView final : public RenderBlockFlow {
45 public: 45 public:
46 explicit RenderView(Document*); 46 explicit RenderView(Document*);
47 virtual ~RenderView(); 47 virtual ~RenderView();
48 virtual void trace(Visitor*) override;
49 void willBeDestroyed() override; 48 void willBeDestroyed() override;
50 49
51 bool hitTest(const HitTestRequest&, HitTestResult&); 50 bool hitTest(const HitTestRequest&, HitTestResult&);
52 bool hitTest(const HitTestRequest&, const HitTestLocation&, HitTestResult&); 51 bool hitTest(const HitTestRequest&, const HitTestLocation&, HitTestResult&);
53 52
54 // Returns the total count of calls to HitTest, for testing. 53 // Returns the total count of calls to HitTest, for testing.
55 unsigned hitTestCount() const { return m_hitTestCount; } 54 unsigned hitTestCount() const { return m_hitTestCount; }
56 55
57 virtual const char* renderName() const override { return "RenderView"; } 56 virtual const char* renderName() const override { return "RenderView"; }
58 57
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 friend class ForceHorriblySlowRectMapping; 182 friend class ForceHorriblySlowRectMapping;
184 183
185 bool shouldUsePrintingLayout() const; 184 bool shouldUsePrintingLayout() const;
186 185
187 RenderObject* backgroundRenderer() const; 186 RenderObject* backgroundRenderer() const;
188 187
189 virtual void invalidateDisplayItemClients(DisplayItemList*) const override; 188 virtual void invalidateDisplayItemClients(DisplayItemList*) const override;
190 189
191 FrameView* m_frameView; 190 FrameView* m_frameView;
192 191
193 RawPtrWillBeMember<RenderObject> m_selectionStart; 192 RenderObject* m_selectionStart;
194 RawPtrWillBeMember<RenderObject> m_selectionEnd; 193 RenderObject* m_selectionEnd;
195 194
196 int m_selectionStartPos; 195 int m_selectionStartPos;
197 int m_selectionEndPos; 196 int m_selectionEndPos;
198 197
199 LayoutUnit m_pageLogicalHeight; 198 LayoutUnit m_pageLogicalHeight;
200 bool m_pageLogicalHeightChanged; 199 bool m_pageLogicalHeightChanged;
201 LayoutState* m_layoutState; 200 LayoutState* m_layoutState;
202 OwnPtr<RenderLayerCompositor> m_compositor; 201 OwnPtr<RenderLayerCompositor> m_compositor;
203 RefPtr<IntervalArena> m_intervalArena; 202 RefPtr<IntervalArena> m_intervalArena;
204 203
205 RawPtrWillBeMember<RenderQuote> m_renderQuoteHead; 204 RenderQuote* m_renderQuoteHead;
206 unsigned m_layoutCounterCount; 205 unsigned m_layoutCounterCount;
207 206
208 unsigned m_hitTestCount; 207 unsigned m_hitTestCount;
209 208
210 class PendingSelection final { 209 class PendingSelection final {
211 DISALLOW_ALLOCATION(); 210 DISALLOW_ALLOCATION();
212 public: 211 public:
213 PendingSelection(); 212 PendingSelection();
214 void setSelection(const FrameSelection&); 213 void setSelection(const FrameSelection&);
215 void clear(); 214 void clear();
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
254 m_paintInvalidationState->m_cachedOffsetsEnabled = true; 253 m_paintInvalidationState->m_cachedOffsetsEnabled = true;
255 } 254 }
256 private: 255 private:
257 const PaintInvalidationState* m_paintInvalidationState; 256 const PaintInvalidationState* m_paintInvalidationState;
258 bool m_didDisable; 257 bool m_didDisable;
259 }; 258 };
260 259
261 } // namespace blink 260 } // namespace blink
262 261
263 #endif // RenderView_h 262 #endif // RenderView_h
OLDNEW
« no previous file with comments | « Source/core/rendering/RenderVTTCue.cpp ('k') | Source/core/rendering/RenderView.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698