diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-06 20:29:18 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-06 20:29:18 +0000 |
| commit | 1ee61486ee0854be7b82d9fca66bb49d563ede4c (patch) | |
| tree | 6f8ba801a5f19e022a281d622d7eaf467d637214 /chrome/browser/plugin_process_host.h | |
| parent | 0928fb62d472e45d2bd23d97aa43b5f704c00bec (diff) | |
| download | chromium_src-1ee61486ee0854be7b82d9fca66bb49d563ede4c.zip chromium_src-1ee61486ee0854be7b82d9fca66bb49d563ede4c.tar.gz chromium_src-1ee61486ee0854be7b82d9fca66bb49d563ede4c.tar.bz2 | |
Do some refactoring of renderer_host.
This removes the last dependency on tab_contents from the renderer_host code
and into the RenderViewHostDelegate. Some of the tests depended on tab_contents,
so I moved to a new directory with the tab_contents include allowed via DEPS.
Now DEPS can enforce that no additional tab_contents includes are added to
renderer_host.
RenderViewHost delegate is now pure virtual. After spending a while *again*
figuring out why my code didn't work, only to find it was because the default
implementation of a function was getting called instead of the real one, I
decided to make this pure virtual. It is implemented by TabContents, which
implements basically everything, and two other places that implement less. Only
two lists of duplicate functions seems not too bad, although long-term it would
be nice if this delegate was somehow more succinct.
Review URL: http://codereview.chromium.org/155071
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19982 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/plugin_process_host.h')
0 files changed, 0 insertions, 0 deletions