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

Issue 429233002: Merge 178823 "Replace further questionable HashMap::add usages i..." (Closed)

Created:
6 years, 4 months ago by jochen (gone - plz use gerrit)
Modified:
6 years, 4 months ago
CC:
blink-reviews, blink-reviews-bindings_chromium.org, arv+blink, abarth-chromium
Project:
blink
Visibility:
Public.

Description

Merge 178823 "Replace further questionable HashMap::add usages i..." > Replace further questionable HashMap::add usages in bindings > > BUG=390928 > R=dcarney@chromium.org > > Review URL: https://codereview.chromium.org/411273002 TBR=jochen@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179238

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M Source/bindings/v8/DOMWrapperMap.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/bindings/v8/SerializedScriptValue.cpp View 3 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jochen (gone - plz use gerrit)
6 years, 4 months ago (2014-07-30 15:31:48 UTC) #1
jochen (gone - plz use gerrit)
Committed patchset #1 manually as r179238.
6 years, 4 months ago (2014-07-30 15:31:54 UTC) #2
haraken
6 years, 4 months ago (2014-07-30 15:32:25 UTC) #3
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld 408576698