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

Issue 701953006: PlzNavigate: Speculatively spawns a renderer process for navigations. (Closed)

Created:
6 years, 1 month ago by carlosk
Modified:
5 years, 11 months ago
CC:
chromium-reviews, darin-cc_chromium.org, nasko+codewatch_chromium.org, jam, creis+watch_chromium.org, ajwong+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Speculatively spawns a renderer process for navigations. Uses a RenderFrameHost instance to hold the reference to the respective SiteInstance and its pre-spawned renderer process. BUG=376094 Committed: https://crrev.com/39337115364bd37da597ae90a023b0a31df9b370 Cr-Commit-Position: refs/heads/master@{#311549}

Patch Set 1 #

Total comments: 34

Patch Set 2 : Addressed creis comments. #

Total comments: 37

Patch Set 3 : Addressed CR comments, added simple test, other minor changes. #

Total comments: 4

Patch Set 4 : Merged back the code paths for RFH creation in RFHM; other related changes. #

Patch Set 5 : PlzNavigate sets the new RFH at commit time; independent of pending_*. #

Patch Set 6 : Reverted SiteInstance creation changes for navigations. #

Patch Set 7 : When navigating checks if the current WebUI can be reused. #

Total comments: 76

Patch Set 8 : Addressed lots of CR comments: mostly renames, moves, reverts and minor changes. #

Patch Set 9 : Uploading the base changes from crrev.com/753173002. #

Patch Set 10 : Re-applying previous changes over. #

Total comments: 22

Patch Set 11 : Uploading the base changes from crrev.com/753173002. #

Patch Set 12 : Re-applied full CL changes, addressed comments, rebased, added tests and fixes. #

Total comments: 1

Patch Set 13 : Minor update to tests: new expects and comments. #

Patch Set 14 : Fixed flakyness of tests that relied on unsafe pointer comparisons. #

Total comments: 15

Patch Set 15 : Address CR comments and fix possible RFH double swap out. #

Total comments: 107

Patch Set 16 : Updated tests and refactored code based on CR comments. #

Total comments: 57

Patch Set 17 : Rebase and changes from CR comments. #

Total comments: 34

Patch Set 18 : Rebase and one more round of CR work. #

Total comments: 35

Patch Set 19 : Rebase. #

Patch Set 20 : Addressed latest round of review comments. #

Patch Set 21 : Fixed missing references to |base| namespace. #

Total comments: 10

Patch Set 22 : Addressed last round of CR comments. #

Total comments: 71

Patch Set 23 : Addressed CR comments, mostly -- but not all -- nits. #

Total comments: 6

Patch Set 24 : Addressed CR comments. #

Total comments: 71

Patch Set 25 : Rebase #

Patch Set 26 : Address CR comments.. #

Total comments: 4

Patch Set 27 : Changes in reply to latest round of comments. #

Patch Set 28 : Latest changes due to review comments. #

Total comments: 72

Patch Set 29 : Addressing latest comments from creis@. #

Total comments: 2

Patch Set 30 : A few minor changes. #

Patch Set 31 : Rebase #

Total comments: 8

Patch Set 32 : Replaced "renderer" to something more meaningful in comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+637 lines, -116 lines) Patch
M content/browser/frame_host/navigator_impl.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 2 chunks +8 lines, -3 lines 0 comments Download
M content/browser/frame_host/navigator_impl_unittest.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 15 chunks +345 lines, -51 lines 0 comments Download
M content/browser/frame_host/render_frame_host_impl.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 1 chunk +4 lines, -0 lines 0 comments Download
M content/browser/frame_host/render_frame_host_manager.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 9 chunks +54 lines, -2 lines 0 comments Download
M content/browser/frame_host/render_frame_host_manager.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 14 chunks +219 lines, -60 lines 0 comments Download
M content/public/test/web_contents_tester.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 1 chunk +2 lines, -0 lines 0 comments Download
M content/test/test_web_contents.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 90 (14 generated)
carlosk
creis, nasko: PTAL pasko, clamy: FYI The core of the logic lives in RenderFrameHostManager. I ...
6 years, 1 month ago (2014-11-05 16:45:45 UTC) #2
carlosk
A couple more comments/questions: * There's a public getter for the pending RFH in RFHM ...
6 years, 1 month ago (2014-11-05 16:54:56 UTC) #3
Charlie Reis
Thanks. I definitely agree that we should have a speculative RFH and not just a ...
6 years, 1 month ago (2014-11-05 17:50:02 UTC) #4
Charlie Reis
Also, please mention PlzNavigate in the CL description.
6 years, 1 month ago (2014-11-05 18:15:09 UTC) #5
carlosk
On 2014/11/05 17:50:02, Charlie Reis wrote: > Thanks. I definitely agree that we should have ...
6 years, 1 month ago (2014-11-05 20:28:59 UTC) #6
Charlie Reis
On 2014/11/05 20:28:59, carlosk wrote: > On 2014/11/05 17:50:02, Charlie Reis wrote: > > Thanks. ...
6 years, 1 month ago (2014-11-06 01:09:10 UTC) #7
carlosk
On 2014/11/06 01:09:10, Charlie Reis wrote: > On 2014/11/05 20:28:59, carlosk wrote: > > On ...
6 years, 1 month ago (2014-11-06 10:56:31 UTC) #8
carlosk
"(...) This is especially true as we move towards Site Isolation, where we may need ...
6 years, 1 month ago (2014-11-06 12:54:17 UTC) #9
nasko
A bunch of comments. I know this is still very much work in progress, but ...
6 years, 1 month ago (2014-11-10 17:35:32 UTC) #10
Charlie Reis
On 2014/11/06 10:56:31, carlosk wrote: > Sounds good. I'll then work on removing the pending_* ...
6 years, 1 month ago (2014-11-11 00:51:28 UTC) #11
clamy
Thanks! A few comments after the first read-through. On the subject of the pending render ...
6 years, 1 month ago (2014-11-12 13:01:14 UTC) #12
nasko
On 2014/11/12 13:01:14, clamy wrote: > Thanks! A few comments after the first read-through. > ...
6 years, 1 month ago (2014-11-12 23:13:11 UTC) #13
carlosk
Thanks for all comments so far. I am still working on a design doc for ...
6 years, 1 month ago (2014-11-13 14:40:26 UTC) #14
carlosk
I just realized there might still be an issue with SiteInstances even with creis@ proposal. ...
6 years, 1 month ago (2014-11-13 14:59:03 UTC) #15
Charlie Reis
On 2014/11/13 14:59:03, carlosk wrote: > I just realized there might still be an issue ...
6 years, 1 month ago (2014-11-13 20:10:41 UTC) #16
clamy
Thanks! A few comments while waiting for the next changes. https://chromiumcodereview.appspot.com/701953006/diff/40001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://chromiumcodereview.appspot.com/701953006/diff/40001/content/browser/frame_host/navigator_impl_unittest.cc#newcode510 ...
6 years, 1 month ago (2014-11-14 10:11:09 UTC) #17
Charlie Reis
On 2014/11/12 23:13:11, nasko wrote: > On 2014/11/12 13:01:14, clamy wrote: > > Thanks! A ...
6 years, 1 month ago (2014-11-14 17:30:29 UTC) #18
Charlie Reis
On 2014/11/13 14:40:26, carlosk wrote: > On 2014/11/11 00:51:28, Charlie Reis wrote: > > On ...
6 years, 1 month ago (2014-11-14 17:39:26 UTC) #19
carlosk
I uploaded a new patch set that reverts the code path split of RFH creation: ...
6 years, 1 month ago (2014-11-14 17:43:37 UTC) #20
carlosk
On 2014/11/14 17:39:26, Charlie Reis wrote: > On 2014/11/13 14:40:26, carlosk wrote: > > On ...
6 years, 1 month ago (2014-11-14 17:49:56 UTC) #21
carlosk
With the last round of PatchSets (4 to 7) I think I got to the ...
6 years, 1 month ago (2014-11-18 15:35:46 UTC) #22
nasko
I think this CL is trying to do more than one thing and ends up ...
6 years, 1 month ago (2014-11-19 01:00:24 UTC) #23
clamy
Thanks! I agree with Nasko that this CL should be split. I think you could ...
6 years, 1 month ago (2014-11-19 15:03:30 UTC) #24
carlosk
On 2014/11/19 01:00:24, nasko wrote: > I think this CL is trying to do more ...
6 years, 1 month ago (2014-11-19 17:24:30 UTC) #25
Charlie Reis
On 2014/11/19 17:24:30, carlosk wrote: > On 2014/11/19 01:00:24, nasko wrote: > > I think ...
6 years, 1 month ago (2014-11-19 17:55:31 UTC) #26
carlosk
On 2014/11/19 17:55:31, Charlie Reis wrote: > On 2014/11/19 17:24:30, carlosk wrote: > > On ...
6 years, 1 month ago (2014-11-19 19:00:27 UTC) #27
nasko
One generic comment - please don't hold back changes locally. It makes reviews much much ...
6 years, 1 month ago (2014-11-19 19:01:29 UTC) #28
carlosk
Thanks. On 2014/11/19 19:01:29, nasko wrote: > One generic comment - please don't hold back ...
6 years, 1 month ago (2014-11-21 14:36:34 UTC) #30
carlosk
Latest two patches allow one to diff this CL against the changes from crrev.com/753173002 by ...
6 years ago (2014-11-24 15:56:19 UTC) #31
nasko
Few more comments. https://codereview.chromium.org/701953006/diff/120001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/120001/content/browser/frame_host/navigator_impl_unittest.cc#newcode307 content/browser/frame_host/navigator_impl_unittest.cc:307: // the pending_* members of RFHM ...
6 years ago (2014-11-25 00:19:11 UTC) #32
clamy
Thanks! A few comments below. https://chromiumcodereview.appspot.com/701953006/diff/200001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://chromiumcodereview.appspot.com/701953006/diff/200001/content/browser/frame_host/navigator_impl_unittest.cc#newcode90 content/browser/frame_host/navigator_impl_unittest.cc:90: TEST_F(NavigatorTestWithBrowserSideNavigation, BeginNavigation) { Is ...
6 years ago (2014-11-25 16:50:57 UTC) #33
carlosk
Thanks for your comments. All done! This upload contains this last round of changes, some ...
6 years ago (2014-11-28 13:08:18 UTC) #36
carlosk
I found some flakyness in one of my tests (on Windows Release) and I think ...
6 years ago (2014-12-01 15:15:27 UTC) #37
clamy
Thanks! A few more comments but it's looking good otherwise. https://chromiumcodereview.appspot.com/701953006/diff/310001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://chromiumcodereview.appspot.com/701953006/diff/310001/content/browser/frame_host/navigator_impl_unittest.cc#newcode249 ...
6 years ago (2014-12-02 12:59:27 UTC) #38
carlosk
Thanks. I addressed all comments but the speculative members naming one. https://codereview.chromium.org/701953006/diff/310001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc (right): ...
6 years ago (2014-12-03 14:31:11 UTC) #39
carlosk
I revisited the DiscardUnusedFrame method based on a previous comment from creis@. https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc ...
6 years ago (2014-12-03 16:25:25 UTC) #42
clamy
Thanks! Once the final comments over the swapped out state are resolved, it should be ...
6 years ago (2014-12-03 17:24:48 UTC) #43
Charlie Reis
Ok, I've started a pass through the CL but haven't gotten to the main code ...
6 years ago (2014-12-04 01:11:32 UTC) #44
Charlie Reis
And here's the rest of the comments for the .cc file! Sorry for the large ...
6 years ago (2014-12-04 21:46:29 UTC) #45
clamy
A few comments as well. Maybe the CL could be split again in two, between ...
6 years ago (2014-12-05 17:16:19 UTC) #46
Charlie Reis
On 2014/12/05 17:16:19, clamy wrote: > A few comments as well. Maybe the CL could ...
6 years ago (2014-12-05 19:06:23 UTC) #47
carlosk
Sorry the extended delay to reply and the large number of comments here. Sickness, trip ...
6 years ago (2014-12-09 07:55:43 UTC) #49
clamy
Thanks! I think it is preferable we postpone moving the RFH swapping in cross-site navigation ...
6 years ago (2014-12-09 15:09:52 UTC) #50
Charlie Reis
Thanks for all the updates. Sounds like we're mainly waiting on the change to commit ...
6 years ago (2014-12-10 22:37:45 UTC) #51
nasko
Few comments as I try to swap back in memory the CL. Thanks for slimming ...
6 years ago (2014-12-10 23:11:34 UTC) #52
ncarter (slow)
https://codereview.chromium.org/701953006/diff/410001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc (right): https://codereview.chromium.org/701953006/diff/410001/content/browser/frame_host/render_frame_host_manager.cc#newcode75 content/browser/frame_host/render_frame_host_manager.cc:75: CleanUpNavigation(); A CL I'm working on -- https://codereview.chromium.org/799593004/ -- ...
6 years ago (2014-12-12 22:37:54 UTC) #54
carlosk
Thanks to all! I hope I replied to all comments but let me know if ...
6 years ago (2014-12-16 01:53:49 UTC) #55
clamy
Thanks! Please find a few comments below. It is looking better, and I think we ...
6 years ago (2014-12-16 15:22:49 UTC) #57
nasko
Few more nits. Camille has raised quite a few points, so I'll wait for the ...
6 years ago (2014-12-18 00:22:40 UTC) #58
carlosk
Thansk for one more round of comments. https://codereview.chromium.org/701953006/diff/410001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc (right): https://codereview.chromium.org/701953006/diff/410001/content/browser/frame_host/render_frame_host_manager.cc#newcode448 content/browser/frame_host/render_frame_host_manager.cc:448: if (render_frame_host ...
6 years ago (2014-12-19 04:27:15 UTC) #60
clamy
Thanks! A few more comments, mostly centered on the tests. https://chromiumcodereview.appspot.com/701953006/diff/430001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://chromiumcodereview.appspot.com/701953006/diff/430001/content/browser/frame_host/navigator_impl_unittest.cc#newcode302 ...
6 years ago (2014-12-19 13:35:57 UTC) #61
Charlie Reis
[Note: Sorry that I didn't get a chance to look at this before going OOO. ...
6 years ago (2014-12-20 00:06:56 UTC) #62
nasko
It is getting better! Hopefully it is close. Few more comments. In general, please separate ...
6 years ago (2014-12-20 00:09:09 UTC) #63
carlosk
Thanks again. Replies and changes are in. Had to upload a following PS because of ...
5 years, 11 months ago (2014-12-29 16:40:17 UTC) #64
clamy
Thanks! A few comments on the latest patch set. https://chromiumcodereview.appspot.com/701953006/diff/490001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc (right): https://chromiumcodereview.appspot.com/701953006/diff/490001/content/browser/frame_host/render_frame_host_manager.cc#newcode664 content/browser/frame_host/render_frame_host_manager.cc:664: ...
5 years, 11 months ago (2014-12-29 17:36:22 UTC) #65
carlosk
Thanks! https://codereview.chromium.org/701953006/diff/490001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc (right): https://codereview.chromium.org/701953006/diff/490001/content/browser/frame_host/render_frame_host_manager.cc#newcode664 content/browser/frame_host/render_frame_host_manager.cc:664: SiteInstance* current_instance = render_frame_host_->GetSiteInstance(); On 2014/12/29 17:36:21, clamy ...
5 years, 11 months ago (2014-12-30 15:54:51 UTC) #68
clamy
Thanks! I think we have the logic right in place, so a lot of comments ...
5 years, 11 months ago (2014-12-30 16:57:46 UTC) #69
carlosk
Indeed mostly nits but there were some pretty useful ones, especially for the tests. Thanks ...
5 years, 11 months ago (2014-12-31 13:39:20 UTC) #70
clamy
Thanks! Lgtm with a few nits. https://chromiumcodereview.appspot.com/701953006/diff/610001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://chromiumcodereview.appspot.com/701953006/diff/610001/content/browser/frame_host/navigator_impl_unittest.cc#newcode90 content/browser/frame_host/navigator_impl_unittest.cc:90: static_cast<MockRenderProcessHost*>(rfh->GetProcess()); On 2014/12/31 ...
5 years, 11 months ago (2014-12-31 14:07:09 UTC) #71
carlosk
Thanks again; all done! https://codereview.chromium.org/701953006/diff/610001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/610001/content/browser/frame_host/navigator_impl_unittest.cc#newcode354 content/browser/frame_host/navigator_impl_unittest.cc:354: TestRenderFrameHost* last_speculative_rfh = On 2014/12/31 ...
5 years, 11 months ago (2014-12-31 15:39:51 UTC) #72
nasko
I have few more changes to review in navigator_impl_unittest.cc, but this should be a good ...
5 years, 11 months ago (2015-01-06 00:03:18 UTC) #73
carlosk
Thanks and PTAL at the replies. https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (left): https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/navigator_impl_unittest.cc#oldcode167 content/browser/frame_host/navigator_impl_unittest.cc:167: SendRequestNavigation(node, kUrl2); On ...
5 years, 11 months ago (2015-01-08 16:05:56 UTC) #74
nasko
https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/navigator_impl_unittest.cc#newcode87 content/browser/frame_host/navigator_impl_unittest.cc:87: // Note: user must call ClearMessages on the sink ...
5 years, 11 months ago (2015-01-08 23:36:37 UTC) #76
carlosk
Thanks once more. Replies and code updates are in. https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/navigator_impl_unittest.cc#newcode182 content/browser/frame_host/navigator_impl_unittest.cc:182: ...
5 years, 11 months ago (2015-01-09 14:48:52 UTC) #77
nasko
I think this is mostly ready to go from my side. Charlie, can you take ...
5 years, 11 months ago (2015-01-09 18:13:07 UTC) #78
carlosk
Thanks. https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/render_frame_host_manager.cc File content/browser/frame_host/render_frame_host_manager.cc (right): https://codereview.chromium.org/701953006/diff/650001/content/browser/frame_host/render_frame_host_manager.cc#newcode729 content/browser/frame_host/render_frame_host_manager.cc:729: DCHECK(new_instance->GetProcess()->HasConnection()); On 2015/01/09 18:13:07, nasko wrote: > On ...
5 years, 11 months ago (2015-01-12 14:35:25 UTC) #79
Charlie Reis
Looks like you all made a lot of progress! Most of my comments are simple ...
5 years, 11 months ago (2015-01-12 23:30:53 UTC) #80
carlosk
Thanks. Changes, replies and a few questions are all up. https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/navigator_impl_unittest.cc#newcode273 ...
5 years, 11 months ago (2015-01-13 15:54:13 UTC) #81
Charlie Reis
Thanks, just a few last things. https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/navigator_impl_unittest.cc#newcode273 content/browser/frame_host/navigator_impl_unittest.cc:273: EXPECT_FALSE(GetSpeculativeRenderFrameHost(rfhm)); On 2015/01/13 ...
5 years, 11 months ago (2015-01-13 18:48:21 UTC) #82
carlosk
Thanks again. All done. https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/navigator_impl_unittest.cc File content/browser/frame_host/navigator_impl_unittest.cc (right): https://codereview.chromium.org/701953006/diff/370001/content/browser/frame_host/navigator_impl_unittest.cc#newcode273 content/browser/frame_host/navigator_impl_unittest.cc:273: EXPECT_FALSE(GetSpeculativeRenderFrameHost(rfhm)); On 2015/01/13 18:48:21, Charlie ...
5 years, 11 months ago (2015-01-14 10:09:47 UTC) #83
Charlie Reis
Thanks! LGTM.
5 years, 11 months ago (2015-01-14 17:59:53 UTC) #84
nasko
Thanks for pulling through this one! LGTM with few minor nits. https://codereview.chromium.org/701953006/diff/810001/content/browser/frame_host/navigator_impl.cc File content/browser/frame_host/navigator_impl.cc (right): ...
5 years, 11 months ago (2015-01-14 19:03:14 UTC) #85
carlosk
Thanks and thanks! Feeling pretty good here! :) https://codereview.chromium.org/701953006/diff/810001/content/browser/frame_host/navigator_impl.cc File content/browser/frame_host/navigator_impl.cc (right): https://codereview.chromium.org/701953006/diff/810001/content/browser/frame_host/navigator_impl.cc#newcode785 content/browser/frame_host/navigator_impl.cc:785: // ...
5 years, 11 months ago (2015-01-14 20:00:48 UTC) #86
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/701953006/830001
5 years, 11 months ago (2015-01-14 20:23:17 UTC) #88
commit-bot: I haz the power
Committed patchset #32 (id:830001)
5 years, 11 months ago (2015-01-14 21:45:31 UTC) #89
commit-bot: I haz the power
5 years, 11 months ago (2015-01-14 21:47:41 UTC) #90
Message was sent while issue was closed.
Patchset 32 (id:??) landed as
https://crrev.com/39337115364bd37da597ae90a023b0a31df9b370
Cr-Commit-Position: refs/heads/master@{#311549}

Powered by Google App Engine
This is Rietveld 408576698