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

Side by Side Diff: third_party/WebKit/Source/core/layout/line/InlineBox.cpp

Issue 2304563003: Add DISABLE_CFI_PERF attribute on the methods with CFI checks disabled. (Closed)
Patch Set: InlineBox Created 4 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2003, 2004, 2005, 2006, 2007 Apple Inc. All rights reserved. 2 * Copyright (C) 2003, 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 #if ENABLE(ASSERT) 53 #if ENABLE(ASSERT)
54 54
55 InlineBox::~InlineBox() 55 InlineBox::~InlineBox()
56 { 56 {
57 if (!m_hasBadParent && m_parent) 57 if (!m_hasBadParent && m_parent)
58 m_parent->setHasBadChildList(); 58 m_parent->setHasBadChildList();
59 } 59 }
60 60
61 #endif 61 #endif
62 62
63 DISABLE_CFI_PERF
63 void InlineBox::destroy() 64 void InlineBox::destroy()
64 { 65 {
65 // We do not need to issue invalidations if the page is being destroyed 66 // We do not need to issue invalidations if the page is being destroyed
66 // since these objects will never be repainted. 67 // since these objects will never be repainted.
67 if (!m_lineLayoutItem.documentBeingDestroyed()) { 68 if (!m_lineLayoutItem.documentBeingDestroyed()) {
68 setLineLayoutItemShouldDoFullPaintInvalidationIfNeeded(); 69 setLineLayoutItemShouldDoFullPaintInvalidationIfNeeded();
69 70
70 // TODO(crbug.com/619630): Make this fast. 71 // TODO(crbug.com/619630): Make this fast.
71 m_lineLayoutItem.slowSetPaintingLayerNeedsRepaint(); 72 m_lineLayoutItem.slowSetPaintingLayerNeedsRepaint();
72 } 73 }
(...skipping 346 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 420
420 void showLineTree(const blink::InlineBox* b) 421 void showLineTree(const blink::InlineBox* b)
421 { 422 {
422 if (b) 423 if (b)
423 b->showLineTreeForThis(); 424 b->showLineTreeForThis();
424 else 425 else
425 fprintf(stderr, "Cannot showLineTree for (nil) InlineBox.\n"); 426 fprintf(stderr, "Cannot showLineTree for (nil) InlineBox.\n");
426 } 427 }
427 428
428 #endif 429 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/line/InlineBox.h ('k') | third_party/WebKit/Source/core/layout/line/InlineFlowBox.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698