Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1164)

Issue 6709038: Move RenderView to content. I've added a new method to ContentRendererClient to determine the te... (Closed)

Created:
9 years, 9 months ago by jam
Modified:
9 years, 6 months ago
Reviewers:
Avi (use Gerrit)
CC:
chromium-reviews, darin-cc_chromium.org, jam, brettw-cc_chromium.org
Visibility:
Public.

Description

Move RenderView to content. I've added a new method to ContentRendererClient to determine the text language. This is only temporary, since the surrounding code should itself be moved out of RenderView. TBR=avi Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=78738

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -6914 lines) Patch
M chrome/chrome_renderer.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/renderer/chrome_content_renderer_client.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/renderer/chrome_content_renderer_client.cc View 3 chunks +29 lines, -0 lines 0 comments Download
M chrome/renderer/render_view.h View 1 chunk +2 lines, -1430 lines 0 comments Download
D chrome/renderer/render_view.cc View 1 chunk +0 lines, -5450 lines 0 comments Download
M content/content_renderer.gypi View 2 chunks +3 lines, -0 lines 0 comments Download
M content/renderer/content_renderer_client.h View 2 chunks +4 lines, -0 lines 0 comments Download
M content/renderer/content_renderer_client.cc View 1 chunk +4 lines, -0 lines 0 comments Download
A + content/renderer/render_view.h View 2 chunks +3 lines, -3 lines 0 comments Download
A + content/renderer/render_view.cc View 4 chunks +3 lines, -29 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jam
9 years, 9 months ago (2011-03-18 19:46:36 UTC) #1
Avi (use Gerrit)
9 years, 9 months ago (2011-03-18 20:14:23 UTC) #2
eyeballed, rubber-stamp lgtm

Powered by Google App Engine
This is Rietveld 408576698