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

Issue 52823006: Small crash fix for set pages dialog drag'n'drop issue. (Closed)

Created:
7 years, 1 month ago by Dan Beam
Modified:
7 years, 1 month ago
Reviewers:
Nico
CC:
chromium-reviews, dbeam+watch-options_chromium.org, dcheng
Visibility:
Public.

Description

Small crash fix for set pages dialog drag'n'drop issue. R=thakis@chromium.org BUG=302284 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232971

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M chrome/browser/custom_home_pages_table_model.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/options/options_browsertest.js View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Dan Beam
this picks up dcheng@'s CL [1] and adds a test for thakis@. the goal is ...
7 years, 1 month ago (2013-11-04 20:42:52 UTC) #1
Nico
lgtm, thanks
7 years, 1 month ago (2013-11-04 20:44:22 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dbeam@chromium.org/52823006/1
7 years, 1 month ago (2013-11-05 00:22:13 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dbeam@chromium.org/52823006/1
7 years, 1 month ago (2013-11-05 03:09:04 UTC) #4
commit-bot: I haz the power
7 years, 1 month ago (2013-11-05 14:04:38 UTC) #5
Message was sent while issue was closed.
Change committed as 232971

Powered by Google App Engine
This is Rietveld 408576698