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

Side by Side Diff: Source/core/html/shadow/SliderThumbElement.cpp

Issue 318803002: Rename Repaint to Paint Invalidation part 3 (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
3 * Copyright (C) 2010 Google Inc. All rights reserved. 3 * Copyright (C) 2010 Google Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
186 else 186 else
187 thumbLocation.setX(thumbLocation.x() - offset); 187 thumbLocation.setX(thumbLocation.x() - offset);
188 thumb->setLocation(thumbLocation); 188 thumb->setLocation(thumbLocation);
189 if (checkForRepaintDuringLayout() && parent() 189 if (checkForRepaintDuringLayout() && parent()
190 && (parent()->style()->appearance() == MediaVolumeSliderPart || parent() ->style()->appearance() == MediaSliderPart)) { 190 && (parent()->style()->appearance() == MediaVolumeSliderPart || parent() ->style()->appearance() == MediaSliderPart)) {
191 // This will sometimes repaint too much. However, it is necessary to 191 // This will sometimes repaint too much. However, it is necessary to
192 // correctly repaint media controls (volume and timeline sliders) - 192 // correctly repaint media controls (volume and timeline sliders) -
193 // they have special painting code in RenderMediaControls.cpp:paintMedia VolumeSlider 193 // they have special painting code in RenderMediaControls.cpp:paintMedia VolumeSlider
194 // and paintMediaSlider that gets called via -webkit-appearance and Rend erTheme, 194 // and paintMediaSlider that gets called via -webkit-appearance and Rend erTheme,
195 // so nothing else would otherwise invalidate the slider. 195 // so nothing else would otherwise invalidate the slider.
196 repaint(); 196 paintInvalidationForWholeRenderer();
197 } 197 }
198 } 198 }
199 199
200 // -------------------------------- 200 // --------------------------------
201 201
202 inline SliderThumbElement::SliderThumbElement(Document& document) 202 inline SliderThumbElement::SliderThumbElement(Document& document)
203 : HTMLDivElement(document) 203 : HTMLDivElement(document)
204 , m_inDragMode(false) 204 , m_inDragMode(false)
205 { 205 {
206 } 206 }
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
467 case MediaVolumeSliderThumbPart: 467 case MediaVolumeSliderThumbPart:
468 case MediaFullScreenVolumeSliderPart: 468 case MediaFullScreenVolumeSliderPart:
469 case MediaFullScreenVolumeSliderThumbPart: 469 case MediaFullScreenVolumeSliderThumbPart:
470 return mediaSliderContainer; 470 return mediaSliderContainer;
471 default: 471 default:
472 return sliderContainer; 472 return sliderContainer;
473 } 473 }
474 } 474 }
475 475
476 } 476 }
OLDNEW
« no previous file with comments | « Source/core/html/shadow/MediaControls.cpp ('k') | Source/core/html/shadow/SpinButtonElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698