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

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

Issue 2758683002: Replace ASSERT_NOT_REACHED with NOTREACHED in core/layout/ (Closed)
Patch Set: Rebase with latest 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) 2008, 2009 Apple Inc. All Rights Reserved. 2 * Copyright (C) 2008, 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 195 matching lines...) Expand 10 before | Expand all | Expand 10 after
206 case ThumbPart: 206 case ThumbPart:
207 return PseudoIdScrollbarThumb; 207 return PseudoIdScrollbarThumb;
208 case TrackBGPart: 208 case TrackBGPart:
209 return PseudoIdScrollbarTrack; 209 return PseudoIdScrollbarTrack;
210 case ScrollbarBGPart: 210 case ScrollbarBGPart:
211 return PseudoIdScrollbar; 211 return PseudoIdScrollbar;
212 case NoPart: 212 case NoPart:
213 case AllParts: 213 case AllParts:
214 break; 214 break;
215 } 215 }
216 ASSERT_NOT_REACHED(); 216 NOTREACHED();
217 return PseudoIdScrollbar; 217 return PseudoIdScrollbar;
218 } 218 }
219 219
220 void LayoutScrollbar::updateScrollbarPart(ScrollbarPart partType, 220 void LayoutScrollbar::updateScrollbarPart(ScrollbarPart partType,
221 bool destroy) { 221 bool destroy) {
222 if (partType == NoPart) 222 if (partType == NoPart)
223 return; 223 return;
224 224
225 RefPtr<ComputedStyle> partStyle = 225 RefPtr<ComputedStyle> partStyle =
226 !destroy 226 !destroy
(...skipping 161 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 } 388 }
389 } 389 }
390 390
391 void LayoutScrollbar::setVisualRect(const LayoutRect& rect) { 391 void LayoutScrollbar::setVisualRect(const LayoutRect& rect) {
392 Scrollbar::setVisualRect(rect); 392 Scrollbar::setVisualRect(rect);
393 for (auto& part : m_parts) 393 for (auto& part : m_parts)
394 part.value->setVisualRect(rect); 394 part.value->setVisualRect(rect);
395 } 395 }
396 396
397 } // namespace blink 397 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutRuby.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutTable.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698