New issue
Advanced search Search tips

Issue 820995 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: ----
Proj-Servicification



Sign in to add a comment

PlatformAppBrowserTest.Restrictions in network_service_browser_tests failing on chromium.linux/Linux Tests (dbg)(1)

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Mar 12 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of dullweber@google.com

PlatformAppBrowserTest.Restrictions in network_service_browser_tests failing on chromium.linux/Linux Tests (dbg)(1)

This test is flaky on network_service_browser_tests:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=PlatformAppBrowserTest.Restrictions&testType=network_service_browser_tests

Builders failed on: 
- Linux Tests (dbg)(1): 
  https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29


 
Components: Internals>Services>Network
Cc: jam@chromium.org
I'm disabling the test: https://crrev.com/c/959014
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 12 2018

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

commit d44013a9c8799892d8738f2c372ab8cd14ab9409
Author: Christian Dullweber <dullweber@chromium.org>
Date: Mon Mar 12 17:00:31 2018

Disable PlatformAppBrowserTest.Restrictions in network_service_browser_tests

The test is flaky.

TBR=jam@chromium.org

Bug:  820995 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I2b0f7baa9491e29184595febf039fb2402358af5
Reviewed-on: https://chromium-review.googlesource.com/959014
Reviewed-by: Christian Dullweber <dullweber@chromium.org>
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542515}
[modify] https://crrev.com/d44013a9c8799892d8738f2c372ab8cd14ab9409/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter

Comment 5 by tapted@chromium.org, Mar 13 2018

Labels: -Sheriff-Chromium
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 3 2018

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

commit 99cf9ba5ad9c5503caa0d83a53d4d389aa8c19b1
Author: arthursonzogni <arthursonzogni@chromium.org>
Date: Tue Apr 03 09:56:16 2018

[Chrome App] Add test: sandboxed page and new window.

Add a new test, it makes use of chrome.app.window.create() with a
sandboxed page. It checks no window is returned to the background app.

There were no tests trying to open a new window with a sandboxed
document already.

Bug:  820995 
Change-Id: I0e59723ee23b31845545774b3ff3dfede0278af0
Reviewed-on: https://chromium-review.googlesource.com/982627
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547657}
[modify] https://crrev.com/99cf9ba5ad9c5503caa0d83a53d4d389aa8c19b1/chrome/browser/apps/app_browsertest.cc
[add] https://crrev.com/99cf9ba5ad9c5503caa0d83a53d4d389aa8c19b1/chrome/test/data/extensions/platform_apps/sandboxed_local_file/main.html
[add] https://crrev.com/99cf9ba5ad9c5503caa0d83a53d4d389aa8c19b1/chrome/test/data/extensions/platform_apps/sandboxed_local_file/manifest.json
[add] https://crrev.com/99cf9ba5ad9c5503caa0d83a53d4d389aa8c19b1/chrome/test/data/extensions/platform_apps/sandboxed_local_file/test.js

Project Member

Comment 7 by bugdroid1@chromium.org, Apr 6 2018

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

commit 725cde7cf9ae7089bf762a6f01c7c0305d4f7c24
Author: arthursonzogni <arthursonzogni@chromium.org>
Date: Fri Apr 06 10:54:51 2018

[Chrome Apps] Test open new window with file not found.

Add a new Chrome Apps test. It tries to open a new window using
chrome.app.window.create() with a file that does not exist.
The callback must be called with createdWindow == undefined.

Bug:  820995 
Change-Id: Iaffe9373e2f19f30d473f3bd165c1a2822f85073
Reviewed-on: https://chromium-review.googlesource.com/986437
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548736}
[modify] https://crrev.com/725cde7cf9ae7089bf762a6f01c7c0305d4f7c24/chrome/browser/apps/app_browsertest.cc
[add] https://crrev.com/725cde7cf9ae7089bf762a6f01c7c0305d4f7c24/chrome/test/data/extensions/platform_apps/new_window_with_non_existing_file/manifest.json
[add] https://crrev.com/725cde7cf9ae7089bf762a6f01c7c0305d4f7c24/chrome/test/data/extensions/platform_apps/new_window_with_non_existing_file/test.js

Project Member

Comment 8 by bugdroid1@chromium.org, May 1 2018

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

commit 45df7dcd718c2e0fdd552499620de7bb7f0b4e6f
Author: Doug Turner <dougt@chromium.org>
Date: Tue May 01 16:58:57 2018

Enabling PlatformAppBrowserTest.Restrictions.

This test was marked flakey.

Bug:  820995 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ie98a919b81cc513c3f56e41921b6cccf4ac0af74
TBR: jam@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1033184
Commit-Queue: Doug Turner <dougt@chromium.org>
Reviewed-by: Doug Turner <dougt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555077}
[modify] https://crrev.com/45df7dcd718c2e0fdd552499620de7bb7f0b4e6f/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter

Comment 9 by dxie@chromium.org, May 15 2018

Labels: -Pri-2 OS-All Proj-Servicification-Canary Pri-1
Owner: dougt@chromium.org
Status: Assigned (was: Available)

Comment 10 by dougt@chromium.org, May 17 2018

Status: Fixed (was: Assigned)

Sign in to add a comment