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

Side by Side Diff: Source/core/platform/chromium/support/WebScrollbarThemeClientImpl.cpp

Issue 26936002: Remove Widget's dependency upon its own inheritor aka ScrollView. This was nasty from an OO design… (Closed) Base URL: https://chromium.googlesource.com/chromium/blink@master
Patch Set: Fix for clang compile error Created 7 years, 2 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) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google 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 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 IntSize WebScrollbarThemeClientImpl::size() const 66 IntSize WebScrollbarThemeClientImpl::size() const
67 { 67 {
68 return m_scrollbar->size(); 68 return m_scrollbar->size();
69 } 69 }
70 70
71 IntPoint WebScrollbarThemeClientImpl::location() const 71 IntPoint WebScrollbarThemeClientImpl::location() const
72 { 72 {
73 return m_scrollbar->location(); 73 return m_scrollbar->location();
74 } 74 }
75 75
76 ScrollView* WebScrollbarThemeClientImpl::parent() const 76 Widget* WebScrollbarThemeClientImpl::parent() const
77 { 77 {
78 // Unused by Chromium scrollbar themes. 78 // Unused by Chromium scrollbar themes.
79 ASSERT_NOT_REACHED(); 79 ASSERT_NOT_REACHED();
80 return 0; 80 return 0;
81 } 81 }
82 82
83 ScrollView* WebScrollbarThemeClientImpl::root() const 83 Widget* WebScrollbarThemeClientImpl::root() const
84 { 84 {
85 // Unused by Chromium scrollbar themes. 85 // Unused by Chromium scrollbar themes.
86 ASSERT_NOT_REACHED(); 86 ASSERT_NOT_REACHED();
87 return 0; 87 return 0;
88 } 88 }
89 89
90 void WebScrollbarThemeClientImpl::setFrameRect(const IntRect&) 90 void WebScrollbarThemeClientImpl::setFrameRect(const IntRect&)
91 { 91 {
92 // Unused by Chromium scrollbar themes. 92 // Unused by Chromium scrollbar themes.
93 ASSERT_NOT_REACHED(); 93 ASSERT_NOT_REACHED();
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
222 { 222 {
223 return m_scrollbar->isAlphaLocked(); 223 return m_scrollbar->isAlphaLocked();
224 } 224 }
225 225
226 void WebScrollbarThemeClientImpl::setIsAlphaLocked(bool flag) 226 void WebScrollbarThemeClientImpl::setIsAlphaLocked(bool flag)
227 { 227 {
228 m_scrollbar->setIsAlphaLocked(flag); 228 m_scrollbar->setIsAlphaLocked(flag);
229 } 229 }
230 230
231 } // namespace WebKit 231 } // namespace WebKit
OLDNEW
« no previous file with comments | « Source/core/platform/chromium/support/WebScrollbarThemeClientImpl.h ('k') | Source/core/rendering/RenderWidget.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698