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

Issue 657450 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

false in inspected-context.cc

Project Member Reported by ClusterFuzz, Oct 19 2016

Issue description

Comment 1 by ajha@chromium.org, Oct 20 2016

Cc: ajha@chromium.org
Labels: M-56
Owner: kozyatinskiy@chromium.org
Status: Assigned (was: Untriaged)
Based on the code search on 'inspected-context.cc' suspecting kozyatinskiy's https://codereview.chromium.org/2413583002.

kozyatinskiy@: Please take a look at this.

Thank you!

Cc: kozyatinskiy@chromium.org
 Issue 658425  has been merged into this issue.
Status: Fixed (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 24 2016

Labels: merge-merged-2883
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bc121c90a1f696b341b88271ad7accf3cb32c6b2

commit bc121c90a1f696b341b88271ad7accf3cb32c6b2
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Mon Oct 24 20:03:42 2016

[DevTools] paint worklet should report context destroyed

If someone reports context to inspector with contextCreated, then this context should be reported as destroyed.

BUG= chromium:657450 , chromium:656626
R=ikilpatrick@chromium.org, dgozman@chromium.org

Review-Url: https://chromiumcodereview.appspot.com/2431223009
Cr-Commit-Position: refs/heads/master@{#426893}
(cherry picked from commit f898cd59b863b70e80d76c254dd62881d0f27f20)

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

Cr-Commit-Position: refs/branch-heads/2883@{#253}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[add] https://crrev.com/bc121c90a1f696b341b88271ad7accf3cb32c6b2/third_party/WebKit/LayoutTests/http/tests/worklet/worklet-should-report-context-destroyed-expected.txt
[add] https://crrev.com/bc121c90a1f696b341b88271ad7accf3cb32c6b2/third_party/WebKit/LayoutTests/http/tests/worklet/worklet-should-report-context-destroyed.html
[modify] https://crrev.com/bc121c90a1f696b341b88271ad7accf3cb32c6b2/third_party/WebKit/Source/modules/csspaint/PaintWorkletGlobalScope.cpp

Project Member

Comment 6 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bc121c90a1f696b341b88271ad7accf3cb32c6b2

commit bc121c90a1f696b341b88271ad7accf3cb32c6b2
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Mon Oct 24 20:03:42 2016

[DevTools] paint worklet should report context destroyed

If someone reports context to inspector with contextCreated, then this context should be reported as destroyed.

BUG= chromium:657450 , chromium:656626
R=ikilpatrick@chromium.org, dgozman@chromium.org

Review-Url: https://chromiumcodereview.appspot.com/2431223009
Cr-Commit-Position: refs/heads/master@{#426893}
(cherry picked from commit f898cd59b863b70e80d76c254dd62881d0f27f20)

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

Cr-Commit-Position: refs/branch-heads/2883@{#253}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[add] https://crrev.com/bc121c90a1f696b341b88271ad7accf3cb32c6b2/third_party/WebKit/LayoutTests/http/tests/worklet/worklet-should-report-context-destroyed-expected.txt
[add] https://crrev.com/bc121c90a1f696b341b88271ad7accf3cb32c6b2/third_party/WebKit/LayoutTests/http/tests/worklet/worklet-should-report-context-destroyed.html
[modify] https://crrev.com/bc121c90a1f696b341b88271ad7accf3cb32c6b2/third_party/WebKit/Source/modules/csspaint/PaintWorkletGlobalScope.cpp

Comment 7 by dimu@google.com, Nov 4 2016

[Automated comment] removing mislabelled merge-merged-2840

Comment 8 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840
Project Member

Comment 9 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label.

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

Sign in to add a comment