New issue
Advanced search Search tips

Issue 608081 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Two SitePerProcessBrowserTest failures on DrMemroy bot

Project Member Reported by osh...@chromium.org, Apr 29 2016

Issue description

SitePerProcessBrowserTest.NavigateRemoteFrameToBlankAndDataURLs
SitePerProcessBrowserTest.RFPHDestruction

are failing on DrMemory bot. It's started on

https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Content%20Browser%20%28DrMemory%29/builds/7152

The following comment seems to be culprit.

https://chromium.googlesource.com/chromium/src/+/9aa6123974ab0f85aecf20cfc6e377071f99b386 (alexmos)

this is less likely, but still possibility
https://chromium.googlesource.com/chromium/src/+/9ab4458be13a0d542768a033fb81286b47615261 (shivanisha)


log:

SitePerProcessBrowserTest.NavigateRemoteFrameToBlankAndDataURLs:

c:\b\build\slave\drm-cr\build\src\content\browser\site_per_process_browsertest.cc(1321): error: Value of: DepictFrameTree(root)
Actual: " Site A ------------ proxies for B\n   |--Site A ------- proxies for B\n   +--Site A ------- proxies for B\n        +--Site A -- proxies for B\nWhere A = http://a.com/\n      B = http://foo.com/"
Expected: " Site A\n" "   |--Site A\n" "   +--Site A\n" "        +--Site A\n" "Where A = http://a.com/"
Which is: " Site A\n   |--Site A\n   +--Site A\n        +--Site A\nWhere A = http://a.com/"
With diff:
@@ -1,5 +1,6 @@
- Site A
-   |--Site A
-   +--Site A
-        +--Site A
+ Site A ------------ proxies for B
+   |--Site A ------- proxies for B
+   +--Site A ------- proxies for B
+        +--Site A -- proxies for B
Where A = http://a.com/
+      B = http://foo.com/

c:\b\build\slave\drm-cr\build\src\content\browser\site_per_process_browsertest.cc(1352): error: Value of: DepictFrameTree(root)
Actual: " Site A ------------ proxies for C\n   |--Site A ------- proxies for C\n   +--Site A ------- proxies for C\n        +--Site A -- proxies for C\nWhere A = http://a.com/\n      C = http://bar.com/"
Expected: " Site A\n" "   |--Site A\n" "   +--Site A\n" "        +--Site A\n" "Where A = http://a.com/"
Which is: " Site A\n   |--Site A\n   +--Site A\n        +--Site A\nWhere A = http://a.com/"
With diff:
@@ -1,5 +1,6 @@
- Site A
-   |--Site A
-   +--Site A
-        +--Site A
+ Site A ------------ proxies for C
+   |--Site A ------- proxies for C
+   +--Site A ------- proxies for C
+        +--Site A -- proxies for C
Where A = http://a.com/
+      C = http://bar.com/


SitePerProcessBrowserTest.RFPHDestruction:

c:\b\build\slave\drm-cr\build\src\content\browser\site_per_process_browsertest.cc(3689): error: Value of: DepictFrameTree(root)
Actual: " Site A ------------ proxies for C\n   |--Site A ------- proxies for C\n   +--Site A ------- proxies for C\n        |--Site A -- proxies for C\n        +--Site A -- proxies for C\n             +--Site A -- proxies for C\nWhere A = http://127.0.0.1/\n      C = http://bar.com/"
Expected: " Site A\n" "   |--Site A\n" "   +--Site A\n" "        |--Site A\n" "        +--Site A\n" "             +--Site A\n" "Where A = http://127.0.0.1/"
Which is: " Site A\n   |--Site A\n   +--Site A\n        |--Site A\n        +--Site A\n             +--Site A\nWhere A = http://127.0.0.1/"
With diff:
@@ -1,7 +1,8 @@
- Site A
-   |--Site A
-   +--Site A
-        |--Site A
-        +--Site A
-             +--Site A
+ Site A ------------ proxies for C
+   |--Site A ------- proxies for C
+   +--Site A ------- proxies for C
+        |--Site A -- proxies for C
+        +--Site A -- proxies for C
+             +--Site A -- proxies for C
Where A = http://127.0.0.1/
+      C = http://bar.com/
 
Cc: creis@chromium.org
Components: Internals>Sandbox>SiteIsolation
Yeah, this is very likely from my r389904.
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 29 2016

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

commit 175b15064ecde59fc52f9760e5ea11b4e2b1f2f3
Author: oshima <oshima@chromium.org>
Date: Fri Apr 29 23:46:34 2016

Exclude failing SitePerProcessBrowserTest on DrMemory bot

BUG= 608081 
TBR=alexmos@chromium.org

Review-Url: https://codereview.chromium.org/1934823002
Cr-Commit-Position: refs/heads/master@{#390806}

[modify] https://crrev.com/175b15064ecde59fc52f9760e5ea11b4e2b1f2f3/tools/valgrind/gtest_exclude/content_browsertests.gtest-drmemory.txt

Project Member

Comment 3 by sheriffbot@chromium.org, Jun 1 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by sheriffbot@chromium.org, Jul 11 2016

Labels: -M-53 MovedFrom-53
This issue has been moved once and is lower than Pri-1. Removing the milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by nasko@chromium.org, Dec 6 2016

Status: WontFix (was: Assigned)
Both of the tests pass with ASan and there is no more DrMemory bot, so closing as WontFix.

Sign in to add a comment