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

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

Issue 2709033003: Migrate WTF::HashMap::get() to ::at() (Closed)
Patch Set: rebase Created 3 years, 9 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) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com) 4 * (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
5 * (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com) 5 * (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
6 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. 6 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc.
7 * All rights reserved. 7 * All rights reserved.
8 * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved. 8 * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 1300 matching lines...) Expand 10 before | Expand all | Expand 10 after
1311 1311
1312 // TODO (lajava) Shouldn't we implement these functions based on physical 1312 // TODO (lajava) Shouldn't we implement these functions based on physical
1313 // direction ?. 1313 // direction ?.
1314 void LayoutBox::clearOverrideContainingBlockContentLogicalHeight() { 1314 void LayoutBox::clearOverrideContainingBlockContentLogicalHeight() {
1315 if (!m_rareData) 1315 if (!m_rareData)
1316 return; 1316 return;
1317 ensureRareData().m_hasOverrideContainingBlockContentLogicalHeight = false; 1317 ensureRareData().m_hasOverrideContainingBlockContentLogicalHeight = false;
1318 } 1318 }
1319 1319
1320 LayoutUnit LayoutBox::extraInlineOffset() const { 1320 LayoutUnit LayoutBox::extraInlineOffset() const {
1321 return gExtraInlineOffsetMap ? gExtraInlineOffsetMap->get(this) 1321 return gExtraInlineOffsetMap ? gExtraInlineOffsetMap->at(this) : LayoutUnit();
1322 : LayoutUnit();
1323 } 1322 }
1324 1323
1325 LayoutUnit LayoutBox::extraBlockOffset() const { 1324 LayoutUnit LayoutBox::extraBlockOffset() const {
1326 return gExtraBlockOffsetMap ? gExtraBlockOffsetMap->get(this) : LayoutUnit(); 1325 return gExtraBlockOffsetMap ? gExtraBlockOffsetMap->at(this) : LayoutUnit();
1327 } 1326 }
1328 1327
1329 void LayoutBox::setExtraInlineOffset(LayoutUnit inlineOffest) { 1328 void LayoutBox::setExtraInlineOffset(LayoutUnit inlineOffest) {
1330 if (!gExtraInlineOffsetMap) 1329 if (!gExtraInlineOffsetMap)
1331 gExtraInlineOffsetMap = new OverrideSizeMap; 1330 gExtraInlineOffsetMap = new OverrideSizeMap;
1332 gExtraInlineOffsetMap->set(this, inlineOffest); 1331 gExtraInlineOffsetMap->set(this, inlineOffest);
1333 } 1332 }
1334 1333
1335 void LayoutBox::setExtraBlockOffset(LayoutUnit blockOffest) { 1334 void LayoutBox::setExtraBlockOffset(LayoutUnit blockOffest) {
1336 if (!gExtraBlockOffsetMap) 1335 if (!gExtraBlockOffsetMap)
(...skipping 4384 matching lines...) Expand 10 before | Expand all | Expand 10 after
5721 5720
5722 void LayoutBox::MutableForPainting:: 5721 void LayoutBox::MutableForPainting::
5723 savePreviousContentBoxSizeAndLayoutOverflowRect() { 5722 savePreviousContentBoxSizeAndLayoutOverflowRect() {
5724 auto& rareData = layoutBox().ensureRareData(); 5723 auto& rareData = layoutBox().ensureRareData();
5725 rareData.m_hasPreviousContentBoxSizeAndLayoutOverflowRect = true; 5724 rareData.m_hasPreviousContentBoxSizeAndLayoutOverflowRect = true;
5726 rareData.m_previousContentBoxSize = layoutBox().contentBoxRect().size(); 5725 rareData.m_previousContentBoxSize = layoutBox().contentBoxRect().size();
5727 rareData.m_previousLayoutOverflowRect = layoutBox().layoutOverflowRect(); 5726 rareData.m_previousLayoutOverflowRect = layoutBox().layoutOverflowRect();
5728 } 5727 }
5729 5728
5730 } // namespace blink 5729 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698