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

Issue 2772403004: Add info on extensions to detect malicious extensions (Closed)

Created:
3 years, 8 months ago by afakhry
Modified:
3 years, 8 months ago
Reviewers:
rkc
CC:
chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Add info on extensions to detect malicious extensions It seems that there are some abusive extensions that act as if they were other extensions with fake names. Instead of recording only the extensions names in feedback reports, we should also record the ID and the version. BUG=699266 TEST=manually Review-Url: https://codereview.chromium.org/2772403004 Cr-Commit-Position: refs/heads/master@{#460514} Committed: https://chromium.googlesource.com/chromium/src/+/dc0d47255880f6b556f5d239ec80d74c8f106f13

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -7 lines) Patch
M chrome/browser/feedback/system_logs/log_sources/chrome_internal_log_source.cc View 2 chunks +12 lines, -7 lines 0 comments Download

Messages

Total messages: 9 (5 generated)
afakhry
Rahul, please take a look. Thanks!
3 years, 8 months ago (2017-03-29 00:39:52 UTC) #3
rkc
lgtm
3 years, 8 months ago (2017-03-29 16:54:38 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2772403004/20001
3 years, 8 months ago (2017-03-29 17:46:04 UTC) #6
commit-bot: I haz the power
3 years, 8 months ago (2017-03-29 20:19:08 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/dc0d47255880f6b556f5d239ec80...

Powered by Google App Engine
This is Rietveld 408576698