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

Issue 469803002: Cleanup: Forward declare MojoApplicationHost in render_process_host_impl.h, instead of including th… (Closed)

Created:
6 years, 4 months ago by viettrungluu
Modified:
6 years, 4 months ago
Reviewers:
sky
CC:
chromium-reviews, darin-cc_chromium.org, nasko+codewatch_chromium.org, jam, creis+watch_chromium.org
Project:
chromium
Visibility:
Public.

Description

Cleanup: Forward declare MojoApplicationHost in render_process_host_impl.h, instead of including the header. R=sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289383

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M content/browser/renderer_host/render_process_host_impl.h View 2 chunks +1 line, -1 line 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
viettrungluu
6 years, 4 months ago (2014-08-13 17:01:57 UTC) #1
sky
LGTM
6 years, 4 months ago (2014-08-13 17:43:20 UTC) #2
viettrungluu
6 years, 4 months ago (2014-08-13 21:26:50 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as 289383 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698