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

Issue 29458575: Issue 4875 - Change app_id parameter names for Microsoft Edge builds (Closed)

Created:
June 7, 2017, 12:01 p.m. by Oleksandr
Modified:
July 7, 2017, 1:39 p.m.
Reviewers:
Sebastian Noack, kzar
Visibility:
Public.

Description

This is meant for the master bookmark of adblockpluschrome

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M metadata.edge View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3
Oleksandr
June 7, 2017, 12:10 p.m. (2017-06-07 12:10:31 UTC) #1
Sebastian Noack
LGTM. This seems to have landed already?
June 7, 2017, 12:27 p.m. (2017-06-07 12:27:46 UTC) #2
Oleksandr
June 7, 2017, 12:29 p.m. (2017-06-07 12:29:55 UTC) #3
On 2017/06/07 12:27:46, Sebastian Noack wrote:
> LGTM. This seems to have landed already?

I am submitting all the patches that are needed for master of adblockpluschrome
to be working on Edge. This is 1 of 6. This has landed on Edge bookmark, but not
on master. (See discussion on IRC).

Powered by Google App Engine
This is Rietveld