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

Unified Diff: content/shell/renderer/test_runner/mock_web_theme_engine_mac.h

Issue 421703003: test_runner: Rename WebTestThemeEngineMac to MockWebThemeEngineMac. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: gyp fix Created 6 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: content/shell/renderer/test_runner/mock_web_theme_engine_mac.h
diff --git a/content/shell/renderer/test_runner/WebTestThemeEngineMac.h b/content/shell/renderer/test_runner/mock_web_theme_engine_mac.h
similarity index 27%
rename from content/shell/renderer/test_runner/WebTestThemeEngineMac.h
rename to content/shell/renderer/test_runner/mock_web_theme_engine_mac.h
index 5b789178cfa277d4261c58cf41921436e77cc742..c5907256f35adfc27046837e1991e4b7a7f04425 100644
--- a/content/shell/renderer/test_runner/WebTestThemeEngineMac.h
+++ b/content/shell/renderer/test_runner/mock_web_theme_engine_mac.h
@@ -2,47 +2,45 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// This implements the WebThemeEngine API in such a way that we match the Mac
-// port rendering more than usual Chromium path, thus allowing us to share
-// more pixel baselines.
-
-#ifndef CONTENT_SHELL_RENDERER_TEST_RUNNER_WEBTESTTHEMEENGINEMAC_H_
-#define CONTENT_SHELL_RENDERER_TEST_RUNNER_WEBTESTTHEMEENGINEMAC_H_
+#ifndef CONTENT_SHELL_RENDERER_TEST_RUNNER_MOCK_WEB_THEME_ENGINE_MAC_H_
+#define CONTENT_SHELL_RENDERER_TEST_RUNNER_MOCK_WEB_THEME_ENGINE_MAC_H_
-#include "base/basictypes.h"
+#include "base/macros.h"
#include "third_party/WebKit/public/platform/WebThemeEngine.h"
namespace content {
-class WebTestThemeEngineMac : public blink::WebThemeEngine {
-public:
- WebTestThemeEngineMac() { }
- virtual ~WebTestThemeEngineMac() { }
-
- virtual void paintScrollbarThumb(
- blink::WebCanvas*,
- blink::WebThemeEngine::State,
- blink::WebThemeEngine::Size,
- const blink::WebRect&,
- const blink::WebThemeEngine::ScrollbarInfo&);
-
-private:
- virtual void paintHIThemeScrollbarThumb(
- blink::WebCanvas*,
- blink::WebThemeEngine::State,
- blink::WebThemeEngine::Size,
- const blink::WebRect&,
- const blink::WebThemeEngine::ScrollbarInfo&);
- virtual void paintNSScrollerScrollbarThumb(
- blink::WebCanvas*,
- blink::WebThemeEngine::State,
- blink::WebThemeEngine::Size,
- const blink::WebRect&,
- const blink::WebThemeEngine::ScrollbarInfo&);
-
- DISALLOW_COPY_AND_ASSIGN(WebTestThemeEngineMac);
+// This implements the WebThemeEngine API in such a way that we match the Mac
+// port rendering more than usual Chromium path, thus allowing us to share
+// more pixel baselines.
+class MockWebThemeEngineMac : public blink::WebThemeEngine {
+ public:
+ MockWebThemeEngineMac() {}
+ virtual ~MockWebThemeEngineMac() {}
+
+ virtual void paintScrollbarThumb(blink::WebCanvas*,
+ blink::WebThemeEngine::State,
+ blink::WebThemeEngine::Size,
+ const blink::WebRect&,
+ const blink::WebThemeEngine::ScrollbarInfo&);
+
+ private:
+ virtual void paintHIThemeScrollbarThumb(
+ blink::WebCanvas*,
+ blink::WebThemeEngine::State,
+ blink::WebThemeEngine::Size,
+ const blink::WebRect&,
+ const blink::WebThemeEngine::ScrollbarInfo&);
+ virtual void paintNSScrollerScrollbarThumb(
+ blink::WebCanvas*,
+ blink::WebThemeEngine::State,
+ blink::WebThemeEngine::Size,
+ const blink::WebRect&,
+ const blink::WebThemeEngine::ScrollbarInfo&);
+
+ DISALLOW_COPY_AND_ASSIGN(MockWebThemeEngineMac);
};
} // namespace content
-#endif // CONTENT_SHELL_RENDERER_TEST_RUNNER_WEBTESTTHEMEENGINEMAC_H_
+#endif // CONTENT_SHELL_RENDERER_TEST_RUNNER_MOCK_WEB_THEME_ENGINE_MAC_H_

Powered by Google App Engine
This is Rietveld 408576698