New issue
Advanced search Search tips

Issue 881605 link

Starred by 1 user
Status: Started
Owner:
vakh@chromium.org
Cc:
vakh@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Remove the use of vector::at() and set::at()

Project Member Reported by vakh@chromium.org, Sep 6 
From: safe_browsing_blocking_page_test.cc, db/v4_store.cc, db/v4_store_unittest.cc, db/v4_database_unittest.cc

Reason: https://groups.google.com/a/chromium.org/d/msg/cxx/6TDgM5g-NH4/NJSmyU01BgAJ

Comment 1 by vakh@chromium.org, Sep 6

Description: Show this description

Comment 2 by vakh@chromium.org, Oct 2

Status: Started (was: Assigned)
safe_browsing_blocking_page_test.cc no longer uses .at()
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 2 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5ab46f4e13a9f02c2a92e1c8c77a7254c64251a6

commit 5ab46f4e13a9f02c2a92e1c8c77a7254c64251a6
Author: Varun Khaneja <vakh@chromium.org>
Date: Tue Oct 02 23:10:59 2018

Remove the usage of .at() from v4_store.cc

Bug: 881605
Change-Id: I55eba01698b7504420cd4c3a6e58f01e98cbad25
Reviewed-on: https://chromium-review.googlesource.com/c/1257811
Reviewed-by: Daniel Rubery <drubery@chromium.org>
Commit-Queue: Varun Khaneja <vakh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596033}
[modify] https://crrev.com/5ab46f4e13a9f02c2a92e1c8c77a7254c64251a6/components/safe_browsing/db/v4_store.cc
►

Sign in to add a comment