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

Unified Diff: ui/compositor/test/in_process_context_factory.cc

Issue 2879463002: Initialize RendererSettings in ContextFactory (Closed)
Patch Set: Fix FakeContextFactory Created 3 years, 7 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: ui/compositor/test/in_process_context_factory.cc
diff --git a/ui/compositor/test/in_process_context_factory.cc b/ui/compositor/test/in_process_context_factory.cc
index d15a998829f0397dd2c5dfea27af3f3a496fb0f2..3283a510fdaf9249f7ab670fd14d42df62f8e776 100644
--- a/ui/compositor/test/in_process_context_factory.cc
+++ b/ui/compositor/test/in_process_context_factory.cc
@@ -11,6 +11,7 @@
#include "base/macros.h"
#include "base/memory/ptr_util.h"
#include "base/threading/thread.h"
+#include "cc/base/switches.h"
#include "cc/output/context_provider.h"
#include "cc/output/output_surface_client.h"
#include "cc/output/output_surface_frame.h"
@@ -29,6 +30,8 @@
#include "ui/compositor/layer.h"
#include "ui/compositor/reflector.h"
#include "ui/compositor/test/in_process_context_provider.h"
+#include "ui/display/display_switches.h"
+#include "ui/gfx/switches.h"
#include "ui/gl/gl_implementation.h"
#include "ui/gl/test/gl_surface_test_support.h"
@@ -139,6 +142,39 @@ InProcessContextFactory::InProcessContextFactory(
DCHECK_NE(gl::GetGLImplementation(), gl::kGLImplementationNone)
<< "If running tests, ensure that main() is calling "
<< "gl::GLSurfaceTestSupport::InitializeOneOff()";
+
+ base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
danakj 2017/05/11 15:17:15 same here regarding tests
Alex Z. 2017/05/11 19:25:22 Done.
+ renderer_settings_.partial_swap_enabled =
+ !command_line->HasSwitch(switches::kUIDisablePartialSwap);
+#if defined(OS_WIN)
+ renderer_settings_.finish_rendering_on_resize = true;
+#elif defined(OS_MACOSX)
+ renderer_settings_.release_overlay_resources_after_gpu_query = true;
+#endif
+ renderer_settings_.gl_composited_texture_quad_border =
+ command_line->HasSwitch(cc::switches::kGlCompositedTextureQuadBorder);
+ renderer_settings_.show_overdraw_feedback =
+ command_line->HasSwitch(cc::switches::kShowOverdrawFeedback);
+ if (command_line->HasSwitch(switches::kUIEnableRGBA4444Textures))
+ renderer_settings_.preferred_tile_format = cc::RGBA_4444;
+ renderer_settings_.enable_color_correct_rendering =
+ command_line->HasSwitch(switches::kEnableColorCorrectRendering) ||
+ command_line->HasSwitch(switches::kEnableHDR);
+ // Populate buffer_to_texture_target_map for all buffer usage/formats.
+ for (int usage_idx = 0; usage_idx <= static_cast<int>(gfx::BufferUsage::LAST);
+ ++usage_idx) {
+ gfx::BufferUsage usage = static_cast<gfx::BufferUsage>(usage_idx);
+ for (int format_idx = 0;
+ format_idx <= static_cast<int>(gfx::BufferFormat::LAST);
+ ++format_idx) {
+ gfx::BufferFormat format = static_cast<gfx::BufferFormat>(format_idx);
+ uint32_t target = GL_TEXTURE_2D;
+ renderer_settings_
+ .buffer_to_texture_target_map[std::make_pair(usage, format)] = target;
+ }
+ }
+ renderer_settings_.disallow_non_exact_resource_reuse =
+ command_line->HasSwitch(cc::switches::kDisallowNonExactResourceReuse);
}
InProcessContextFactory::~InProcessContextFactory() {
@@ -214,11 +250,11 @@ void InProcessContextFactory::CreateCompositorFrameSink(
display_output_surface->capabilities().max_frames_pending));
data->display = base::MakeUnique<cc::Display>(
- &shared_bitmap_manager_, &gpu_memory_buffer_manager_,
- compositor->GetRendererSettings(), compositor->frame_sink_id(),
- begin_frame_source.get(), std::move(display_output_surface),
- std::move(scheduler), base::MakeUnique<cc::TextureMailboxDeleter>(
- compositor->task_runner().get()));
+ &shared_bitmap_manager_, &gpu_memory_buffer_manager_, renderer_settings_,
+ compositor->frame_sink_id(), begin_frame_source.get(),
+ std::move(display_output_surface), std::move(scheduler),
+ base::MakeUnique<cc::TextureMailboxDeleter>(
+ compositor->task_runner().get()));
// Note that we are careful not to destroy a prior |data->begin_frame_source|
// until we have reset |data->display|.
data->begin_frame_source = std::move(begin_frame_source);
@@ -275,12 +311,6 @@ double InProcessContextFactory::GetRefreshRate() const {
return refresh_rate_;
}
-uint32_t InProcessContextFactory::GetImageTextureTarget(
- gfx::BufferFormat format,
- gfx::BufferUsage usage) {
- return GL_TEXTURE_2D;
-}
-
gpu::GpuMemoryBufferManager*
InProcessContextFactory::GetGpuMemoryBufferManager() {
return &gpu_memory_buffer_manager_;
@@ -312,6 +342,11 @@ void InProcessContextFactory::ResizeDisplay(ui::Compositor* compositor,
per_compositor_data_[compositor]->display->Resize(size);
}
+const cc::RendererSettings& InProcessContextFactory::GetRendererSettings()
+ const {
+ return renderer_settings_;
+}
+
void InProcessContextFactory::AddObserver(ContextFactoryObserver* observer) {
observer_list_.AddObserver(observer);
}
« ui/compositor/test/fake_context_factory.cc ('K') | « ui/compositor/test/in_process_context_factory.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698