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

Side by Side Diff: content/browser/renderer_host/compositor_impl_android.cc

Issue 1900713006: Remove the android browser compositor context from global share group. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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 | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/compositor_impl_android.h" 5 #include "content/browser/renderer_host/compositor_impl_android.h"
6 6
7 #include <android/bitmap.h> 7 #include <android/bitmap.h>
8 #include <android/native_window_jni.h> 8 #include <android/native_window_jni.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 506 matching lines...) Expand 10 before | Expand all | Expand 10 after
517 BrowserGpuChannelHostFactory::instance(); 517 BrowserGpuChannelHostFactory::instance();
518 // This channel might be lost (and even if it isn't right now, it might 518 // This channel might be lost (and even if it isn't right now, it might
519 // still get marked as lost from the IO thread, at any point in time really). 519 // still get marked as lost from the IO thread, at any point in time really).
520 // But from here on just try and always lead to either 520 // But from here on just try and always lead to either
521 // DidInitializeOutputSurface() or DidFailToInitializeOutputSurface(). 521 // DidInitializeOutputSurface() or DidFailToInitializeOutputSurface().
522 scoped_refptr<gpu::GpuChannelHost> gpu_channel_host(factory->GetGpuChannel()); 522 scoped_refptr<gpu::GpuChannelHost> gpu_channel_host(factory->GetGpuChannel());
523 523
524 GURL url("chrome://gpu/CompositorImpl::CreateOutputSurface"); 524 GURL url("chrome://gpu/CompositorImpl::CreateOutputSurface");
525 gpu::SurfaceHandle surface_handle = 525 gpu::SurfaceHandle surface_handle =
526 GpuSurfaceTracker::Get()->GetSurfaceHandle(surface_id_); 526 GpuSurfaceTracker::Get()->GetSurfaceHandle(surface_id_);
527 constexpr bool share_resources = true; 527 constexpr bool share_resources = false;
528 constexpr bool automatic_flushes = false; 528 constexpr bool automatic_flushes = false;
529 529
530 gpu::SharedMemoryLimits limits; 530 gpu::SharedMemoryLimits limits;
531 // This limit is meant to hold the contents of the display compositor 531 // This limit is meant to hold the contents of the display compositor
532 // drawing the scene. See discussion here: 532 // drawing the scene. See discussion here:
533 // https://codereview.chromium.org/1900993002/diff/90001/content/browser/rende rer_host/compositor_impl_android.cc?context=3&column_width=80&tab_spaces=8 533 // https://codereview.chromium.org/1900993002/diff/90001/content/browser/rende rer_host/compositor_impl_android.cc?context=3&column_width=80&tab_spaces=8
534 limits.command_buffer_size = 64 * 1024; 534 limits.command_buffer_size = 64 * 1024;
535 // These limits are meant to hold the uploads for the browser UI without 535 // These limits are meant to hold the uploads for the browser UI without
536 // any excess space. 536 // any excess space.
537 limits.start_transfer_buffer_size = 64 * 1024; 537 limits.start_transfer_buffer_size = 64 * 1024;
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
656 656
657 void CompositorImpl::SetNeedsAnimate() { 657 void CompositorImpl::SetNeedsAnimate() {
658 needs_animate_ = true; 658 needs_animate_ = true;
659 if (!host_->visible()) 659 if (!host_->visible())
660 return; 660 return;
661 661
662 host_->SetNeedsAnimate(); 662 host_->SetNeedsAnimate();
663 } 663 }
664 664
665 } // namespace content 665 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698