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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 10551002: TabContentsWrapper -> TabContents, part 56. (Closed)

Created:
8 years, 6 months ago by Avi (use Gerrit)
Modified:
8 years, 6 months ago
Reviewers:
jam
CC:
chromium-reviews, yoshiki+watch_chromium.org, dcheng, dmazzoni+watch_chromium.org, ajwong+watch_chromium.org, cbentzel+watch_chromium.org, aboxhall+watch_chromium.org, kkania, apatrick_chromium, darin-cc_chromium.org, brettw-cc_chromium.org, tim (not reviewing), Avi (use Gerrit), jennb, creis+watch_chromium.org, Raghu Simha, ncarter (slow), yuzo+watch_chromium.org, jianli, ctguil+watch_chromium.org, zork+watch_chromium.org, mihaip-chromium-reviews_chromium.org, akalin, hashimoto+watch_chromium.org, Dmitry Titov, davidbarr+watch_chromium.org, Aaron Boodman, robertshield, dtseng+watch_chromium.org
Visibility:
Public.

Description

TabContentsWrapper -> TabContents, part 56. That's almost all. BUG=131026 TEST=no change Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=141891

Patch Set 1 #

Patch Set 2 : merging in part 52 #

Patch Set 3 : Not killing GetSelectedWebContents yet #

Unified diffs Side-by-side diffs Delta from patch set Stats (+109 lines, -160 lines) Patch
M chrome/browser/accessibility/accessibility_extension_api.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/accessibility/accessibility_extension_apitest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/accessibility/accessibility_win_browsertest.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/automation/automation_provider_observers.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/custom_handlers/protocol_handler_registry_browsertest.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/extensions/extension_install_ui_android.h View 1 chunk +0 lines, -3 lines 0 comments Download
M chrome/browser/first_run/first_run.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/net/load_timing_observer_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/page_cycler/page_cycler.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/page_cycler/page_cycler_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/safe_browsing/client_side_detection_host_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/glue/synced_tab_delegate.h View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/browser/ui/browser.h View 1 2 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/ui/browser.cc View 1 2 2 chunks +0 lines, -10 lines 0 comments Download
M chrome/browser/ui/omnibox/omnibox_view_browsertest.cc View 3 chunks +6 lines, -6 lines 0 comments Download
M chrome/browser/ui/panels/old_base_panel_browser_test.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M chrome/browser/ui/panels/old_panel_browsertest.cc View 3 chunks +4 lines, -5 lines 0 comments Download
M chrome/browser/ui/pdf/pdf_browsertest.cc View 10 chunks +15 lines, -15 lines 0 comments Download
M chrome/browser/ui/sync/browser_synced_window_delegate.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/tab_contents/core_tab_helper.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/tab_contents/tab_contents.h View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/browser/ui/tab_contents/tab_contents.cc View 1 chunk +0 lines, -12 lines 0 comments Download
M chrome/browser/ui/tab_contents/tab_contents_iterator.cc View 1 chunk +1 line, -1 line 0 comments Download
D chrome/browser/ui/tab_contents/tab_contents_wrapper.h View 1 chunk +0 lines, -13 lines 0 comments Download
M chrome/browser/ui/views/frame/browser_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/constrained_web_dialog_ui_browsertest.cc View 4 chunks +14 lines, -13 lines 0 comments Download
M chrome/browser/ui/zoom/zoom_controller.h View 1 3 chunks +3 lines, -4 lines 0 comments Download
M chrome/browser/ui/zoom/zoom_controller.cc View 1 4 chunks +7 lines, -8 lines 0 comments Download
M chrome/browser/ui/zoom/zoom_observer.h View 1 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/test/base/browser_with_test_window_test.cc View 2 chunks +2 lines, -3 lines 0 comments Download
M chrome/test/base/in_process_browser_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/base/test_browser_window.h View 4 chunks +4 lines, -4 lines 0 comments Download
M chrome/test/base/ui_test_utils.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/test/base/ui_test_utils.cc View 9 chunks +12 lines, -12 lines 0 comments Download
M chrome/test/gpu/gpu_pixel_browsertest.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/test/perf/rendering/latency_tests.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/test/perf/rendering/throughput_tests.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/test/ui/ppapi_uitest.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Avi (use Gerrit)
8 years, 6 months ago (2012-06-12 23:00:43 UTC) #1
jam
8 years, 6 months ago (2012-06-12 23:41:54 UTC) #2 
lgtm

Powered by Google App Engine
This is Rietveld 408576698