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

Unified Diff: mojo/services/html_viewer/webthemeengine_impl.cc

Issue 808553007: Move HTMLViewer out of mojo namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix merge mistake in test Created 6 years 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
« no previous file with comments | « mojo/services/html_viewer/webthemeengine_impl.h ('k') | mojo/services/html_viewer/webthread_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/services/html_viewer/webthemeengine_impl.cc
diff --git a/mojo/services/html_viewer/webthemeengine_impl.cc b/mojo/services/html_viewer/webthemeengine_impl.cc
index 24906e5bc7c416ffaf40fadb5b7f0b460e9640c8..c91462cb92bbcfd21cccb2ca3d89b390023a1971 100644
--- a/mojo/services/html_viewer/webthemeengine_impl.cc
+++ b/mojo/services/html_viewer/webthemeengine_impl.cc
@@ -14,10 +14,10 @@ using blink::WebColor;
using blink::WebRect;
using blink::WebThemeEngine;
-namespace mojo {
+namespace html_viewer {
+namespace {
-static ui::NativeTheme::Part NativeThemePart(
- WebThemeEngine::Part part) {
+ui::NativeTheme::Part NativeThemePart(WebThemeEngine::Part part) {
switch (part) {
case WebThemeEngine::PartScrollbarDownArrow:
return ui::NativeTheme::kScrollbarDownArrow;
@@ -60,8 +60,7 @@ static ui::NativeTheme::Part NativeThemePart(
}
}
-static ui::NativeTheme::State NativeThemeState(
- WebThemeEngine::State state) {
+ui::NativeTheme::State NativeThemeState(WebThemeEngine::State state) {
switch (state) {
case WebThemeEngine::StateDisabled:
return ui::NativeTheme::kDisabled;
@@ -76,7 +75,7 @@ static ui::NativeTheme::State NativeThemeState(
}
}
-static void GetNativeThemeExtraParams(
+void GetNativeThemeExtraParams(
WebThemeEngine::Part part,
WebThemeEngine::State state,
const WebThemeEngine::ExtraParams* extra_params,
@@ -160,6 +159,8 @@ static void GetNativeThemeExtraParams(
}
}
+} // namespace
+
blink::WebSize WebThemeEngineImpl::getSize(WebThemeEngine::Part part) {
ui::NativeTheme::ExtraParams extra;
return ui::NativeTheme::instance()->GetPartSize(NativeThemePart(part),
@@ -199,4 +200,4 @@ void WebThemeEngineImpl::paintStateTransition(blink::WebCanvas* canvas,
gfx::Rect(rect));
}
-} // namespace mojo
+} // namespace html_viewer
« no previous file with comments | « mojo/services/html_viewer/webthemeengine_impl.h ('k') | mojo/services/html_viewer/webthread_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698