Skip to content

GitLab

T
tangled
Switch branch/tag
  1. 05 Aug, 2019 36 commits
  3. 04 Aug, 2019 4 commits
    • chromium-internal-autoroll's avatar
      Roll src-internal 80d451499877..f7a047eea7e8 (1 commits) · 31c1304a
      chromium-internal-autoroll authored 
      https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/80d451499877..f7a047eea7e8


Created with:
  gclient setdep -r src-internal@f7a047eea7e8

The AutoRoll server is located here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome
TBR=jbudorick@google.com

Bug: chromium:None
Change-Id: I04d6e7ef2c71a317fa64bb77eb75dd62cfc90d00
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1735719Reviewed-by: default avatarchromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#683889}
      31c1304a
    • Noel Gordon's avatar
      Revert "[Locked Fullscreen] Fix the vox panel inconsistency with the fullscreen mode." · 7a14a3ef
      Noel Gordon authored 
      This reverts commit 8d9e0bcb.

Reason for revert: ScreenUtilTest.FullscreenWindowBoundsWithChromeVox
is crashing linux-chromeos-dbg. Example build:

https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/13699

Crash log:

https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8906215673747190784/+/steps/ash_unittests/0/logs/Deterministic_failure:_ScreenUtilTest.FullscreenWindowBoundsWithChromeVox__status_CRASH_/0




Original change's description:
> [Locked Fullscreen] Fix the vox panel inconsistency with the fullscreen mode.
> 
> 
> Upon entering the fullscreen/locked-fullscreen while the chromeVox panel is working, You can see an empty bar in the top of the screen.
> 1. The fullscreen rect y dimension starts after the height of the chromeVox panel, and as we don't want the chromeVox panel to be displayed, its height should be removed from the calculations.
> 
> UPDATE: This issue got fixed by another colleague.
> So I added a unittest, and removed two workarounds were added to fix this issue in the locked-fullscreen mode.
> (one was to disable chromevox panel upon starting the lockedscreen mode, and the other was to hide the vox panel specifically for the locked fullscreen mode)
> 
> When a window is set to fullscreen screen_util::GetFullscreenWindowBoundsInParent & AccessibilityPanelLayoutManager::UpdateWindowBounds() are called twice with this stacktraces order:
> 
> [23272:23272:0723/164119.757770:ERROR:screen_util.cc(47)] ayaaa #0 0x0000081162bc base::debug::CollectStackTrace()
> #1 0x0000080a6c88 base::debug::StackTrace::StackTrace()
> #2 0x000009a8b57c ash::screen_util::GetFullscreenWindowBoundsInParent()
> #3 0x000009b93e62 ash::wm::DefaultState::UpdateBoundsFromState()
> #4 0x000009b93da4 ash::wm::DefaultState::EnterToNextState()
> #5 0x000009b93c2e ash::wm::DefaultState::HandleTransitionEvents()
> #6 0x000009b92e3e ash::wm::WindowState::OnWindowPropertyChanged()
> #7 0x000008d085f6 aura::Window::AfterPropertyChange()
> #8 0x0000082c62fc ui::PropertyHandler::SetPropertyInternal()
> #9 0x000008d0324c ui::PropertyHandler::SetProperty<>()
> #10 0x000009268956 views::Widget::SetFullscreen()
> #11 0x00000a088c28 BrowserView::ProcessFullscreen()
> #12 0x000009f48638 FullscreenController::EnterFullscreenModeInternal()
> #13 0x000009f47c98 FullscreenController::ToggleFullscreenModeInternal()
> #14 0x000009f47bba FullscreenController::ToggleBrowserFullscreenMode()
> #15 0x00000a0b45a8 (anonymous namespace)::BrowserWindowStateDelegate::ToggleFullscreen()
> #16 0x000009b93fc0 ash::wm::ToggleFullScreen()
> #17 0x000009b9177e ash::wm::WindowState::OnWMEvent()
> #18 0x000009a153ca ash::accelerators::ToggleFullscreen()
> #19 0x000009a13f72 ash::(anonymous namespace)::HandleToggleFullscreen()
> #20 0x000009a12332 ash::AcceleratorControllerImpl::AcceleratorPressed()
> #21 0x000009253988 ui::AcceleratorManager::Process()
> #22 0x000009ac225a ash::PreTargetAcceleratorHandler::ProcessAccelerator()
> ....
> #80 0x0000eae970a2 __libc_start_main
> 
> [23272:23272:0723/164121.271202:ERROR:accessibility_panel_layout_manager.cc(103)] ayaaa #0 0x0000081162bc base::debug::CollectStackTrace()
> #1 0x0000080a6c88 base::debug::StackTrace::StackTrace()
> #2 0x000009a190fa ash::AccessibilityPanelLayoutManager::UpdateWindowBounds()
> #3 0x0000099ebd12 display::DisplayManager::NotifyMetricsChanged()
> #4 0x0000099ebf5a display::DisplayManager::UpdateWorkAreaOfDisplay()
> #5 0x000009a33610 ash::WindowTreeHostManager::UpdateWorkAreaOfDisplayNearestWindow()
> #6 0x000009a94ec6 ash::ShelfLayoutManager::UpdateBoundsAndOpacity()
> #7 0x000009a93680 ash::ShelfLayoutManager::SetState()
> #8 0x000009a932be ash::ShelfLayoutManager::UpdateVisibilityState()
> #9 0x000009b96428 ash::WorkspaceLayoutManager::SetChildBounds()
> #10 0x000009b8f6e4 ash::wm::SetBoundsInScreen()
> #11 0x000009276346 views::NativeWidgetAura::SetBounds()
> #12 0x000009225b72 views::BubbleDialogDelegateView::SizeToContents()
> #13 0x000009269980 views::Widget::OnNativeWidgetMove()
> #14 0x000009276cd4 views::NativeWidgetAura::OnBoundsChanged()
> #15 0x00000a0b4332 BrowserFrameAsh::OnBoundsChanged()
> #16 0x000008d0978c aura::Window::OnLayerBoundsChanged()
> #17 0x000008d19a6e ui::Layer::SetBoundsFromAnimation()
> #18 0x000008d0730c aura::Window::SetBoundsInternal()
> #19 0x000009b929a6 ash::wm::WindowState::SetBoundsDirect()
> #20 0x000009b93f0e ash::wm::DefaultState::UpdateBoundsFromState()
> #21 0x000009b93da4 ash::wm::DefaultState::EnterToNextState()
> #22 0x000009b93c2e ash::wm::DefaultState::HandleTransitionEvents()
> #23 0x000009b92e3e ash::wm::WindowState::OnWindowPropertyChanged()
> #24 0x000008d085f6 aura::Window::AfterPropertyChange()
> #25 0x0000082c62fc ui::PropertyHandler::SetPropertyInternal()
> #26 0x000008d0324c ui::PropertyHandler::SetProperty<>()
> #27 0x000009268956 views::Widget::SetFullscreen()
> #28 0x00000a088c28 BrowserView::ProcessFullscreen()
> #29 0x000009f48638 FullscreenController::EnterFullscreenModeInternal()
> #30 0x000009f47c98 FullscreenController::ToggleFullscreenModeInternal()
> #31 0x000009f47bba FullscreenController::ToggleBrowserFullscreenMode()
> #32 0x00000a0b45a8 (anonymous namespace)::BrowserWindowStateDelegate::ToggleFullscreen()
> #33 0x000009b93fc0 ash::wm::ToggleFullScreen()
> #34 0x000009b9177e ash::wm::WindowState::OnWMEvent()
> #35 0x000009a153ca ash::accelerators::ToggleFullscreen()
> #36 0x000009a13f72 ash::(anonymous namespace)::HandleToggleFullscreen()
> #37 0x000009a12332 ash::AcceleratorControllerImpl::AcceleratorPressed()
> #38 0x000009253988 ui::AcceleratorManager::Process()
> #39 0x000009ac225a ash::PreTargetAcceleratorHandler::ProcessAccelerator()
> #40 0x000009ab4ebe wm::AcceleratorFilter::OnKeyEvent()
> ....
> #97 0x0000eae970a2 __libc_start_main
> 
> [23272:23272:0723/164123.029099:ERROR:screen_util.cc(47)] ayaaa #0 0x0000081162bc base::debug::CollectStackTrace()
> #1 0x0000080a6c88 base::debug::StackTrace::StackTrace()
> #2 0x000009a8b57c ash::screen_util::GetFullscreenWindowBoundsInParent()
> #3 0x000009b93764 ash::wm::DefaultState::SetMaximizedOrFullscreenBounds()
> #4 0x000009b9367a ash::wm::DefaultState::HandleWorkspaceEvents()
> #5 0x000009b5ac5e ash::wm::BaseState::OnWMEvent()
> #6 0x000009b9177e ash::wm::WindowState::OnWMEvent()
> #7 0x000009b968e6 ash::WorkspaceLayoutManager::AdjustAllWindowsBoundsForWorkAreaChange()
> #8 0x000009b96af8 ash::WorkspaceLayoutManager::OnDisplayMetricsChanged()
> #9 0x0000099ebd12 display::DisplayManager::NotifyMetricsChanged()
> #10 0x0000099ebf5a display::DisplayManager::UpdateWorkAreaOfDisplay()
> #11 0x000009a33610 ash::WindowTreeHostManager::UpdateWorkAreaOfDisplayNearestWindow()
> #12 0x000009a94ec6 ash::ShelfLayoutManager::UpdateBoundsAndOpacity()
> #13 0x000009a93680 ash::ShelfLayoutManager::SetState()
> #14 0x000009a932be ash::ShelfLayoutManager::UpdateVisibilityState()
> #15 0x000009b96428 ash::WorkspaceLayoutManager::SetChildBounds()
> #16 0x000009b8f6e4 ash::wm::SetBoundsInScreen()
> #17 0x000009276346 views::NativeWidgetAura::SetBounds()
> #18 0x000009225b72 views::BubbleDialogDelegateView::SizeToContents()
> #19 0x000009269980 views::Widget::OnNativeWidgetMove()
> #20 0x000009276cd4 views::NativeWidgetAura::OnBoundsChanged()
> #21 0x00000a0b4332 BrowserFrameAsh::OnBoundsChanged()
> #22 0x000008d0978c aura::Window::OnLayerBoundsChanged()
> #23 0x000008d19a6e ui::Layer::SetBoundsFromAnimation()
> #24 0x000008d0730c aura::Window::SetBoundsInternal()
> #25 0x000009b929a6 ash::wm::WindowState::SetBoundsDirect()
> #26 0x000009b93f0e ash::wm::DefaultState::UpdateBoundsFromState()
> #27 0x000009b93da4 ash::wm::DefaultState::EnterToNextState()
> #28 0x000009b93c2e ash::wm::DefaultState::HandleTransitionEvents()
> #29 0x000009b92e3e ash::wm::WindowState::OnWindowPropertyChanged()
> #30 0x000008d085f6 aura::Window::AfterPropertyChange()
> #31 0x0000082c62fc ui::PropertyHandler::SetPropertyInternal()
> #32 0x000008d0324c ui::PropertyHandler::SetProperty<>()
> #33 0x000009268956 views::Widget::SetFullscreen()
> #34 0x00000a088c28 BrowserView::ProcessFullscreen()
> #35 0x000009f48638 FullscreenController::EnterFullscreenModeInternal()
> #36 0x000009f47c98 FullscreenController::ToggleFullscreenModeInternal()
> #37 0x000009f47bba FullscreenController::ToggleBrowserFullscreenMode()
> #38 0x00000a0b45a8 (anonymous namespace)::BrowserWindowStateDelegate::ToggleFullscreen()
> #39 0x000009b93fc0 ash::wm::ToggleFullScreen()
> #40 0x000009b9177e ash::wm::WindowState::OnWMEvent()
> #41 0x000009a153ca ash::accelerators::ToggleFullscreen()
> #42 0x000009a13f72 ash::(anonymous namespace)::HandleToggleFullscreen()
> #43 0x000009a12332 ash::AcceleratorControllerImpl::AcceleratorPressed()
> #44 0x000009253988 ui::AcceleratorManager::Process()
> #45 0x000009ac225a ash::PreTargetAcceleratorHandler::ProcessAccelerator()
> #46 0x000009ab4ebe wm::AcceleratorFilter::OnKeyEvent()
> ....
> #103 0x0000eae970a2 __libc_start_main
> 
> [23272:23272:0723/164124.932334:ERROR:accessibility_panel_layout_manager.cc(103)] ayaaa #0 0x0000081162bc base::debug::CollectStackTrace()
> #1 0x0000080a6c88 base::debug::StackTrace::StackTrace()
> #2 0x000009a190fa ash::AccessibilityPanelLayoutManager::UpdateWindowBounds()
> #3 0x000009ab0dea ash::Shell::NotifyFullscreenStateChanged()
> #4 0x000009b96a4c ash::WorkspaceLayoutManager::OnPostWindowStateTypeChange()
> #5 0x000009b92738 ash::wm::WindowState::NotifyPostStateTypeChange()
> #6 0x000009b93ddc ash::wm::DefaultState::EnterToNextState()
> #7 0x000009b93c2e ash::wm::DefaultState::HandleTransitionEvents()
> #8 0x000009b92e3e ash::wm::WindowState::OnWindowPropertyChanged()
> #9 0x000008d085f6 aura::Window::AfterPropertyChange()
> #10 0x0000082c62fc ui::PropertyHandler::SetPropertyInternal()
> #11 0x000008d0324c ui::PropertyHandler::SetProperty<>()
> #12 0x000009268956 views::Widget::SetFullscreen()
> #13 0x00000a088c28 BrowserView::ProcessFullscreen()
> #14 0x000009f48638 FullscreenController::EnterFullscreenModeInternal()
> #15 0x000009f47c98 FullscreenController::ToggleFullscreenModeInternal()
> #16 0x000009f47bba FullscreenController::ToggleBrowserFullscreenMode()
> #17 0x00000a0b45a8 (anonymous namespace)::BrowserWindowStateDelegate::ToggleFullscreen()
> #18 0x000009b93fc0 ash::wm::ToggleFullScreen()
> #19 0x000009b9177e ash::wm::WindowState::OnWMEvent()
> #20 0x000009a153ca ash::accelerators::ToggleFullscreen()
> #21 0x000009a13f72 ash::(anonymous namespace)::HandleToggleFullscreen()
> #22 0x000009a12332 ash::AcceleratorControllerImpl::AcceleratorPressed()
> #23 0x000009253988 ui::AcceleratorManager::Process()
> #24 0x000009ac225a ash::PreTargetAcceleratorHandler::ProcessAccelerator()
> #25 0x000009ab4ebe wm::AcceleratorFilter::OnKeyEvent()
> ...
> #82 0x0000eae970a2 __libc_start_main
> 
> Bug: 945794,959786
> Change-Id: If04f70f2c71563e9660c18ca942ef4b798ebcd3a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1702312
> Commit-Queue: Aya Elsayed <ayaelattar@google.com>
> Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
> Reviewed-by: Aga Wronska <agawronska@chromium.org>
> Reviewed-by: Ivan Šandrk <isandrk@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#683631}

TBR=oshima@chromium.org,dtseng@chromium.org,isandrk@chromium.org,agawronska@chromium.org,ayaelattar@google.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 945794, 959786
Change-Id: Iff0e849ffbba12b8a36079c2ba484e9285d75780
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1735720Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#683888}
      7a14a3ef
    • chromium-autoroll's avatar
      Roll Fuchsia SDK from 890606035392 to 890603479401 · e60b59f6
      chromium-autoroll authored 
      The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast
TBR=cr-fuchsia+bot@chromium.org

Change-Id: Icc0fc18b7a7175fce4df4a1542db285f4d7d196c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1735786Reviewed-by: default avatarchromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#683887}
      e60b59f6
    • Mike Reed's avatar
      remove guard flag to enable new bicubic filtering · 90f7b1a7
      Mike Reed authored 
      Change-Id: I76cdaa8506ce43e38f15e835cb5c14aa5d4bcea5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1734746Reviewed-by: default avatarFlorin Malita <fmalita@chromium.org>
Commit-Queue: Mike Reed <reed@google.com>
Cr-Commit-Position: refs/heads/master@{#683886}
      90f7b1a7