New issue
Advanced search Search tips

Issue 889790 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: ExtensionWebstorePrivateApiTest.InstallAccepted



Sign in to add a comment

ExtensionWebstorePrivateApiTest.InstallAccepted is flaky

Project Member Reported by Findit, Sep 27

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Sep 27

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

commit 50db3d6ab7f88f43fcf010513859959ddf291555
Author: Marina Ciocea <marinaciocea@chromium.org>
Date: Thu Sep 27 13:10:44 2018

Revert "Download Shelf : Connect OfflineItem to the shelf"

This reverts commit e6f5eba3b2be98ba20ead82a46e4f7963a71111d.

Reason for revert: looks related to the Win10 Test flakiness reported by findit.

Original change's description:
> Download Shelf : Connect OfflineItem to the shelf
> 
> This CL connects offline item with the download shelf. Some of
> the OfflineItemModel methods are still not implemented.
> 
> Bug: 881499
> Change-Id: I2f0cd71815a08bcddfca753378543749389d78db
> Reviewed-on: https://chromium-review.googlesource.com/1239875
> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
> Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
> Reviewed-by: Min Qin <qinmin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#594471}

TBR=qinmin@chromium.org,carlosk@chromium.org,shaktisahu@chromium.org

Change-Id: I98129d5eaca139d27ab7bb15f3ca194efb1331c6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 881499, 889790
Reviewed-on: https://chromium-review.googlesource.com/1249068
Reviewed-by: Marina Ciocea <marinaciocea@chromium.org>
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594697}
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_core_service_impl.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_core_service_impl.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_item_model.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_item_model.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_shelf.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_shelf.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_shelf_controller.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_shelf_controller.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_shelf_unittest.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/download_ui_controller.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/offline_item_model.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/offline_item_model.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/offline_item_model_manager_factory.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/offline_item_model_manager_factory.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/offline_item_utils.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/offline_item_utils.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/test_download_shelf.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/download/test_download_shelf.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/offline_pages/android/offline_page_bridge.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/ui/views/download/download_shelf_view.cc
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/chrome/browser/ui/views/download/download_shelf_view.h
[modify] https://crrev.com/50db3d6ab7f88f43fcf010513859959ddf291555/components/offline_items_collection/core/launch_location.h

Labels: -Sheriff-Chromium
Owner: shaktisahu@chromium.org
Status: Assigned (was: Untriaged)
Shakti, please have a look 

Sign in to add a comment