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

Issue 6732027: Replace include with forward declarations.... (Closed)

Created:
9 years, 9 months ago by cary
Modified:
9 years, 7 months ago
Reviewers:
brettw, caryclark, reed1
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Replace include with forward declarations. Remove #import ApplicationServices.h from platform_device_mac.h, and add it back to appropriate .cc files. This is motivated by a conflict between headers included by ApplicationServices.h and WebCore headers. The conflict occurs in WebMediaPlayerClientImpl.cpp if it modified to include platform_canvas.h. With this change, ReleaseBitmapContext() cannot be declared inline in the header, so it is moved to bitmap_platform_device_mac.cc. There is no functional change. This requires that http://trac.webkit.org/changeset/81979 land in the Chromium tree first. (81979 has landed in the WebKit tree.) BUG=none TEST=none

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -6 lines) Patch
M skia/ext/bitmap_platform_device_data.h View 1 chunk +1 line, -5 lines 0 comments Download
M skia/ext/bitmap_platform_device_mac.cc View 2 chunks +7 lines, -0 lines 0 comments Download
M skia/ext/platform_canvas_unittest.cc View 1 1 chunk +2 lines, -0 lines 0 comments Download
M skia/ext/platform_device_mac.h View 1 chunk +3 lines, -1 line 0 comments Download
M skia/ext/platform_device_mac.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
brettw
LGTM
9 years, 9 months ago (2011-03-24 19:50:46 UTC) #1
cary
9 years, 9 months ago (2011-03-24 19:57:41 UTC) #2
reed1
I'll try locally installing this, and running the try bots http://codereview.chromium.org/6732027/diff/1/skia/ext/platform_device_mac.h File skia/ext/platform_device_mac.h (right): http://codereview.chromium.org/6732027/diff/1/skia/ext/platform_device_mac.h#newcode12 ...
9 years, 9 months ago (2011-03-24 20:18:54 UTC) #3
cary
WebNetscapePluginEventHandler.h and WebKitSystemInterface.h use struct CGRect; IntRect.h and FloatRect.h use typedef struct CGRect CGRect; I ...
9 years, 9 months ago (2011-03-24 20:34:26 UTC) #4
caryclark
The WebKit dependency has landed, and the patch above successfully builds and runs. What's needed ...
9 years, 8 months ago (2011-04-07 19:03:06 UTC) #5
brettw
9 years, 8 months ago (2011-04-07 23:42:09 UTC) #6
You just need to harass somebody to check it in for you. I can do it when I come
in tomorrow if it hasn't been done by then.

Powered by Google App Engine
This is Rietveld 408576698