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

Issue 600014 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

bluetooth/characteristicvaluechanged.html crashes if layout tests run with leak detection

Project Member Reported by lukasza@chromium.org, Apr 1 2016

Issue description

The crash started to happen in https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/builds/18539

Unfortunately, so far I was not able to repro the problem locally.
 
Owner: ortuno@chromium.org
ortuno@, could you please take a look? (I see that you've modified bluetooth/characteristicvaluechanged.html yesterday in https://codereview.chromium.org/1815483003)
Labels: -Pri-3 Pri-1
Bumping up priority, since this problem makes one of the bots red.
Status: Started (was: Untriaged)
We are in the process of refactoring our tests. I believe that will fix the problem.
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 4 2016

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

commit 47e15c538f5c24300e6a02bd71016ba70ae5a0a3
Author: ortuno <ortuno@chromium.org>
Date: Mon Apr 04 20:18:21 2016

bluetooth: Remove disconnect when page hidden

Also removes the permission check when disconnecting. This stops the renderer
crashing when the renderer tries to disconnect a device after a test has
finished and the browser state has been cleared.

BUG= 554240 , 600014 , 579746 

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

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

[modify] https://crrev.com/47e15c538f5c24300e6a02bd71016ba70ae5a0a3/content/browser/bluetooth/bluetooth_dispatcher_host.cc
[modify] https://crrev.com/47e15c538f5c24300e6a02bd71016ba70ae5a0a3/third_party/WebKit/LayoutTests/bluetooth/connect.html
[delete] https://crrev.com/276dc49243e685b85adbadf2f63decc0509bd3c3/third_party/WebKit/LayoutTests/bluetooth/disconnect-when-hidden-or-closed.html
[modify] https://crrev.com/47e15c538f5c24300e6a02bd71016ba70ae5a0a3/third_party/WebKit/LayoutTests/bluetooth/resources/bluetooth-helpers.js
[modify] https://crrev.com/47e15c538f5c24300e6a02bd71016ba70ae5a0a3/third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.cpp
[modify] https://crrev.com/47e15c538f5c24300e6a02bd71016ba70ae5a0a3/third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h
[modify] https://crrev.com/47e15c538f5c24300e6a02bd71016ba70ae5a0a3/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTServer.cpp

Status: Fixed (was: Started)

Comment 6 by sshru...@google.com, May 18 2016

Labels: Test-Layout

Sign in to add a comment