Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29787568: Issue 5987 - Removes ext.windows (Closed)

Created:
May 23, 2018, 1:09 a.m. by Jon Sonesen
Modified:
May 23, 2018, 11:09 p.m.
Reviewers:
kzar
CC:
Sebastian Noack
Visibility:
Public.

Description

Issue 5987 - Removes ext.windows

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -12 lines) Patch
M ext/background.js View 1 chunk +0 lines, -12 lines 0 comments Download

Messages

Total messages: 3
Jon Sonesen
May 23, 2018, 1:09 a.m. (2018-05-23 01:09:42 UTC) #1
kzar
Please could you add a link to this review to the issue and mark it ...
May 23, 2018, 12:01 p.m. (2018-05-23 12:01:53 UTC) #2
Jon Sonesen
May 23, 2018, 11:06 p.m. (2018-05-23 23:06:51 UTC) #3
On 2018/05/23 12:01:53, kzar wrote:
> Please could you add a link to this review to the issue and mark it as under
> review?
> 
> Anyway the change itself LGTM

 Done, thanks

Powered by Google App Engine
This is Rietveld