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

Side by Side Diff: Source/testing/runner/WebTestThemeControlWin.h

Issue 23440025: TestRunner library classes should be marked as NonCopyable. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 years, 3 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
« no previous file with comments | « Source/testing/runner/WebPermissions.h ('k') | Source/testing/runner/WebTestThemeEngineMac.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google Inc. All rights reserved. 2 * Copyright (C) 2010 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 are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * 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 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 21 matching lines...) Expand all
32 // needed by WebThemeEngineDRTWin. See the comments in that class 32 // needed by WebThemeEngineDRTWin. See the comments in that class
33 // header file for why this class is needed and used. 33 // header file for why this class is needed and used.
34 // 34 //
35 // This class implements a generic set of widgets using Skia. The widgets 35 // This class implements a generic set of widgets using Skia. The widgets
36 // are optimized for testability, not a pleasing appearance. 36 // are optimized for testability, not a pleasing appearance.
37 // 37 //
38 38
39 #ifndef WebTestThemeControlWin_h 39 #ifndef WebTestThemeControlWin_h
40 #define WebTestThemeControlWin_h 40 #define WebTestThemeControlWin_h
41 41
42 #include "public/platform/WebNonCopyable.h"
42 #include "third_party/skia/include/core/SkColor.h" 43 #include "third_party/skia/include/core/SkColor.h"
43 #include "third_party/skia/include/core/SkRect.h" 44 #include "third_party/skia/include/core/SkRect.h"
44 45
45 // Skia forward declarations 46 // Skia forward declarations
46 class SkCanvas; 47 class SkCanvas;
47 48
48 namespace WebTestRunner { 49 namespace WebTestRunner {
49 50
50 class WebTestThemeControlWin { 51 class WebTestThemeControlWin : public WebKit::WebNonCopyable {
51 public: 52 public:
52 // This list of states mostly mirrors the list in WebCore/platform/ThemeType s.h 53 // This list of states mostly mirrors the list in WebCore/platform/ThemeType s.h
53 // but is maintained separately since that isn't public and also to minimize 54 // but is maintained separately since that isn't public and also to minimize
54 // dependencies. 55 // dependencies.
55 // Note that the WebKit ThemeTypes seem to imply that a control can be 56 // Note that the WebKit ThemeTypes seem to imply that a control can be
56 // in multiple states simultaneously but WebThemeEngine only allows for 57 // in multiple states simultaneously but WebThemeEngine only allows for
57 // a single state at a time. 58 // a single state at a time.
58 // 59 //
59 // Some definitions for the various states: 60 // Some definitions for the various states:
60 // Disabled - indicates that a control can't be modified or selected 61 // Disabled - indicates that a control can't be modified or selected
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
191 const int m_right; 192 const int m_right;
192 const int m_top; 193 const int m_top;
193 const int m_bottom; 194 const int m_bottom;
194 const int m_width; 195 const int m_width;
195 const int m_height; 196 const int m_height;
196 }; 197 };
197 198
198 } 199 }
199 200
200 #endif // WebTestThemeControlWin_h 201 #endif // WebTestThemeControlWin_h
OLDNEW
« no previous file with comments | « Source/testing/runner/WebPermissions.h ('k') | Source/testing/runner/WebTestThemeEngineMac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698