Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(247)

Issue 81433004: DevTools: Pass foreign bit to Blink instead of owner_pid (Closed)

Created:
7 years, 1 month ago by alph
Modified:
7 years, 1 month ago
CC:
chromium-reviews, vsevik, jam, yurys, joi+watch-content_chromium.org, paulirish+reviews_chromium.org, darin-cc_chromium.org, devtools-reviews_chromium.org, piman+watch_chromium.org, aandrey+blink_chromium.org, pfeldman
Visibility:
Public.

Description

DevTools: Pass foreign bit to Blink instead of owner_pid BUG=298951 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=236778

Patch Set 1 #

Patch Set 2 : Reupload #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -5 lines) Patch
M content/common/devtools_messages.h View 1 chunk +1 line, -1 line 0 comments Download
M content/common/gpu/devtools_gpu_agent.h View 2 chunks +2 lines, -1 line 0 comments Download
M content/common/gpu/devtools_gpu_agent.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/renderer/devtools/devtools_agent.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
alph
Antoine, could you please take a look. Julien, I've made an one-liner type change in ...
7 years, 1 month ago (2013-11-21 22:18:54 UTC) #1
jln (very slow on Chromium)
On 2013/11/21 22:18:54, alph wrote: > Antoine, could you please take a look. > > ...
7 years, 1 month ago (2013-11-21 22:21:02 UTC) #2
alph-g
Fixed already. Sorry about that. On Thu, Nov 21, 2013 at 2:21 PM, <jln@chromium.org> wrote: ...
7 years, 1 month ago (2013-11-21 22:21:40 UTC) #3
piman
lgtm
7 years, 1 month ago (2013-11-21 22:57:02 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alph@chromium.org/81433004/100001
7 years, 1 month ago (2013-11-22 01:22:06 UTC) #5
commit-bot: I haz the power
Retried try job too often on linux_rel for step(s) browser_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&number=193633
7 years, 1 month ago (2013-11-22 03:11:19 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alph@chromium.org/81433004/100001
7 years, 1 month ago (2013-11-22 04:03:36 UTC) #7
commit-bot: I haz the power
Retried try job too often on linux_rel for step(s) browser_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&number=193748
7 years, 1 month ago (2013-11-22 06:21:50 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alph@chromium.org/81433004/100001
7 years, 1 month ago (2013-11-22 15:40:59 UTC) #9
commit-bot: I haz the power
7 years, 1 month ago (2013-11-22 16:32:21 UTC) #10
Message was sent while issue was closed.
Change committed as 236778

Powered by Google App Engine
This is Rietveld 408576698