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

Side by Side Diff: third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp

Issue 2517653002: Convert a few more CFI blacklist entries into DISABLE_CFI_PERF attributes. (Closed)
Patch Set: Revert some of the changes which have a high risk of breaking things Created 4 years, 1 month 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) 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2009 Apple 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 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 1007 matching lines...) Expand 10 before | Expand all | Expand 10 after
1018 // FrameView::paintRecursively() does so. 1018 // FrameView::paintRecursively() does so.
1019 m_layer->layer()->invalidate(); 1019 m_layer->layer()->invalidate();
1020 for (size_t i = 0; i < m_linkHighlights.size(); ++i) 1020 for (size_t i = 0; i < m_linkHighlights.size(); ++i)
1021 m_linkHighlights[i]->invalidate(); 1021 m_linkHighlights[i]->invalidate();
1022 getPaintController().invalidateAll(); 1022 getPaintController().invalidateAll();
1023 1023
1024 trackRasterInvalidation(*this, IntRect(IntPoint(), expandedIntSize(m_size)), 1024 trackRasterInvalidation(*this, IntRect(IntPoint(), expandedIntSize(m_size)),
1025 PaintInvalidationFull); 1025 PaintInvalidationFull);
1026 } 1026 }
1027 1027
1028 DISABLE_CFI_PERF
1028 void GraphicsLayer::setNeedsDisplayInRect( 1029 void GraphicsLayer::setNeedsDisplayInRect(
1029 const IntRect& rect, 1030 const IntRect& rect,
1030 PaintInvalidationReason invalidationReason, 1031 PaintInvalidationReason invalidationReason,
1031 const DisplayItemClient& client) { 1032 const DisplayItemClient& client) {
1032 if (!drawsContent()) 1033 if (!drawsContent())
1033 return; 1034 return;
1034 1035
1035 m_layer->layer()->invalidateRect(rect); 1036 m_layer->layer()->invalidateRect(rect);
1036 if (firstPaintInvalidationTrackingEnabled()) 1037 if (firstPaintInvalidationTrackingEnabled())
1037 m_debugInfo.appendAnnotatedInvalidateRect(rect, invalidationReason); 1038 m_debugInfo.appendAnnotatedInvalidateRect(rect, invalidationReason);
(...skipping 272 matching lines...) Expand 10 before | Expand all | Expand 10 after
1310 void showGraphicsLayerTree(const blink::GraphicsLayer* layer) { 1311 void showGraphicsLayerTree(const blink::GraphicsLayer* layer) {
1311 if (!layer) { 1312 if (!layer) {
1312 LOG(INFO) << "Cannot showGraphicsLayerTree for (nil)."; 1313 LOG(INFO) << "Cannot showGraphicsLayerTree for (nil).";
1313 return; 1314 return;
1314 } 1315 }
1315 1316
1316 String output = layer->layerTreeAsText(blink::LayerTreeIncludesDebugInfo); 1317 String output = layer->layerTreeAsText(blink::LayerTreeIncludesDebugInfo);
1317 LOG(INFO) << output.utf8().data(); 1318 LOG(INFO) << output.utf8().data();
1318 } 1319 }
1319 #endif 1320 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698