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

Issue 10389228: browser: Rename message_box_handler.* to javascript_dialog_creator.* (Closed)

Created:
8 years, 7 months ago by tfarina
Modified:
8 years, 7 months ago
CC:
chromium-reviews, Aaron Boodman, mihaip-chromium-reviews_chromium.org, dcheng
Visibility:
Public.

Description

browser: Rename message_box_handler.* to javascript_dialog_creator.* So the filename matches much more with the free function names used in the header file and with the class name used in the source file. R=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138398

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -330 lines) Patch
M chrome/browser/extensions/extension_host.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/external_tab/external_tab_container_win.cc View 1 chunk +1 line, -1 line 0 comments Download
A + chrome/browser/ui/app_modal_dialogs/javascript_dialog_creator.h View 2 chunks +3 lines, -3 lines 0 comments Download
A + chrome/browser/ui/app_modal_dialogs/javascript_dialog_creator.cc View 7 chunks +19 lines, -19 lines 0 comments Download
D chrome/browser/ui/app_modal_dialogs/message_box_handler.h View 1 chunk +0 lines, -25 lines 0 comments Download
D chrome/browser/ui/app_modal_dialogs/message_box_handler.cc View 1 chunk +0 lines, -276 lines 0 comments Download
M chrome/browser/ui/browser.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/gtk/tabs/dragged_tab_controller_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/views/tabs/tab_drag_controller.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tfarina
8 years, 7 months ago (2012-05-19 21:55:19 UTC) #1
Ben Goodger (Google)
8 years, 7 months ago (2012-05-21 15:42:07 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698