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

Issue 78713006: ozone: Fix embedded component build with enable_printing==0 (Closed)

Created:
7 years, 1 month ago by spang
Modified:
7 years, 1 month ago
CC:
chromium-reviews, epoger
Visibility:
Public.

Description

ozone: Fix embedded component build with enable_printing==0 PDF support is removed from skia without printing, but VectorPlatformDeviceSkia inherits from SkPDFDevice. We need to remove VectorPlatformDeviceSkia as well. This fixes the following undefined symbol errors. lib/libskia.so: error: undefined reference to 'SkPDFDevice::~SkPDFDevice()' lib/libskia.so: error: undefined reference to 'SkPDFDevice::SkPDFDevice(SkTSize<int> const&, SkTSize<int> const&, SkMatrix const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::getDeviceCapabilities()' lib/libskia.so: error: undefined reference to 'SkPDFDevice::onAttachToCanvas(SkCanvas*)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::onDetachFromCanvas()' lib/libskia.so: error: undefined reference to 'SkPDFDevice::clear(unsigned int)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawPaint(SkDraw const&, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawPoints(SkDraw const&, SkCanvas::PointMode, unsigned long, SkPoint const*, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawRect(SkDraw const&, SkRect const&, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawRRect(SkDraw const&, SkRRect const&, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawPath(SkDraw const&, SkPath const&, SkPaint const&, SkMatrix const*, bool)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawBitmap(SkDraw const&, SkBitmap const&, SkMatrix const&, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawSprite(SkDraw const&, SkBitmap const&, int, int, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawBitmapRect(SkDraw const&, SkBitmap const&, SkRect const*, SkRect const&, SkPaint const&, SkCanvas::DrawBitmapRectFlags)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawText(SkDraw const&, void const*, unsigned long, float, float, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawPosText(SkDraw const&, void const*, unsigned long, float const*, float, int, SkPa int const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawTextOnPath(SkDraw const&, void const*, unsigned long, SkPath const&, SkMatrix con st*, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawVertices(SkDraw const&, SkCanvas::VertexMode, int, SkPoint const*, SkPoint const* , unsigned int const*, SkXfermode*, unsigned short const*, int, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::drawDevice(SkDraw const&, SkBaseDevice*, int, int, SkPaint const&)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::onReadPixels(SkBitmap const&, int, int, SkCanvas::Config8888)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::allowImageFilter(SkImageFilter*)' lib/libskia.so: error: undefined reference to 'SkPDFDevice::onCreateCompatibleDevice(SkBitmap::Config, int, int, bool, SkBaseDevice::Usage)' TEST=build with GYP_DEFINES="embedded==1 component=shared_library" Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=236813

Patch Set 1 #

Patch Set 2 : split vector_platform_device_skia.cc into new condition #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M skia/skia_chrome.gypi View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
spang
7 years, 1 month ago (2013-11-20 19:06:32 UTC) #1
epoger
Seems fine to me, but vandebo knows this stuff much better than I do. Steve?
7 years, 1 month ago (2013-11-20 19:13:03 UTC) #2
vandebo (ex-Chrome)
LGTM
7 years, 1 month ago (2013-11-22 17:28:12 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/78713006/70001
7 years, 1 month ago (2013-11-22 18:47:07 UTC) #4
commit-bot: I haz the power
7 years, 1 month ago (2013-11-22 19:56:01 UTC) #5
Message was sent while issue was closed.
Change committed as 236813

Powered by Google App Engine
This is Rietveld 408576698