New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 598792 link

Starred by 2 users
Status: Fixed
Owner:
█ fsam...@chromium.org
Last visit > 30 days ago
Closed: Mar 2016
Cc:
danakj@chromium.org
█ vollick@chromium.org
penghuang@chromium.org
█ markdittmer@chromium.org
rjkroege@chromium.org
vmi...@chromium.org
sadrul@chromium.org
piman@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug
mus



Sign in to add a comment

DelegatedFrameHost, ResizeLock, BrowserCompositorMac => content/browser/renderer_host

Project Member Reported by fsam...@chromium.org, Mar 29 2016 
In the future, we will likely move content/browser/compositor to components/display_compositor to reuse that code in Mus.

Some classes are very content specific:

- DelegatedFrameHost
- ResizeLock
- BrowserCompositorMac

We can move those to content/browser/renderer_host
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 29 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6168175d0420c55052f79fe1113c8270c163cd8f

commit 6168175d0420c55052f79fe1113c8270c163cd8f
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Mar 29 22:29:19 2016

BrowserCompositorMac, ResizeLock, DelegatedFrameHost => content/browser/renderer_host

In the future, content/browser/compositor will likely be
moved to components/display_compositor. In anticipation of
that move, I'm moving files that are content specific to
content/browser/renderer_host.

BUG= 598792 

Review URL: https://codereview.chromium.org/1841153002

Cr-Commit-Position: refs/heads/master@{#383850}

[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/browser_main_loop.cc
[rename] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/browser_compositor_view_mac.h
[rename] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/browser_compositor_view_mac.mm
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/compositor_resize_lock_aura.h
[rename] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/delegated_frame_host.cc
[rename] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/delegated_frame_host.h
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/render_widget_host_view_aura.h
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/render_widget_host_view_mac.h
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/render_widget_host_view_mac.mm
[rename] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/resize_lock.cc
[rename] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/browser/renderer_host/resize_lock.h
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/content_browser.gypi
[modify] https://crrev.com/6168175d0420c55052f79fe1113c8270c163cd8f/content/test/content_browser_test_utils_internal.cc

Comment 2 by fsam...@chromium.org, Mar 31 2016

Owner: fsam...@chromium.org
Status: Fixed (was: Untriaged)
►

Sign in to add a comment