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

Issue 29793570: Issue 6692 - Update frame structure for about:blank frames earlier (Closed)

Created:
May 29, 2018, 4:45 p.m. by kzar
Modified:
May 30, 2018, 10:46 a.m.
Reviewers:
Sebastian Noack
Visibility:
Public.

Description

Issue 6692 - Update frame structure for about:blank frames earlier

Patch Set 1 #

Total comments: 2

Patch Set 2 : Only handle about:blank #

Total comments: 4

Patch Set 3 : Addressed nits #

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

Messages

Total messages: 6
kzar
Patch Set 1
May 29, 2018, 4:47 p.m. (2018-05-29 16:47:42 UTC) #1
Sebastian Noack
https://codereview.adblockplus.org/29793570/diff/29793571/ext/background.js File ext/background.js (right): https://codereview.adblockplus.org/29793570/diff/29793571/ext/background.js#newcode268 ext/background.js:268: if (!(url.startsWith("http:") || Wouldn't checking for "about:blank" be sufficient ...
May 29, 2018, 4:48 p.m. (2018-05-29 16:48:49 UTC) #2
kzar
Patch Set 2 : Only handle about:blank https://codereview.adblockplus.org/29793570/diff/29793571/ext/background.js File ext/background.js (right): https://codereview.adblockplus.org/29793570/diff/29793571/ext/background.js#newcode268 ext/background.js:268: if (!(url.startsWith("http:") ...
May 29, 2018, 5:28 p.m. (2018-05-29 17:28:12 UTC) #3
Sebastian Noack
https://codereview.adblockplus.org/29793570/diff/29793578/ext/background.js File ext/background.js (right): https://codereview.adblockplus.org/29793570/diff/29793578/ext/background.js#newcode280 ext/background.js:280: // order to properly attribute the requests they make. ...
May 29, 2018, 5:35 p.m. (2018-05-29 17:35:09 UTC) #4
kzar
Patch Set 3 : Addressed nits https://codereview.adblockplus.org/29793570/diff/29793578/ext/background.js File ext/background.js (right): https://codereview.adblockplus.org/29793570/diff/29793578/ext/background.js#newcode280 ext/background.js:280: // order to ...
May 29, 2018, 5:46 p.m. (2018-05-29 17:46:18 UTC) #5
Sebastian Noack
May 29, 2018, 5:56 p.m. (2018-05-29 17:56:34 UTC) #6
LGTM

Powered by Google App Engine
This is Rietveld