Open
Bug 932274
Opened 11 years ago
Updated 2 years ago
Use Moz2D size classes in ipdl
Categories
(Core :: General, defect)
Core
General
Tracking
()
NEW
People
(Reporter: dzbarsky, Unassigned)
Details
Attachments
(1 file)
31.28 KB,
patch
|
nical
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Updated•11 years ago
|
Attachment #823983 -
Attachment is patch: true
Attachment #823983 -
Attachment mime type: message/rfc822 → text/plain
Attachment #823983 -
Flags: review?(nical.bugzilla)
Comment 1•11 years ago
|
||
David, can you put the comments into the bug as to why we're doing this? I'm not suggesting we don't have a good reason, just that we should write it down so that it's obvious a few years from now. Thanks.
Updated•11 years ago
|
Attachment #823983 -
Flags: review?(nical.bugzilla) → review+
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to Milan Sreckovic [:milan] from comment #1) > David, can you put the comments into the bug as to why we're doing this? I'm > not suggesting we don't have a good reason, just that we should write it > down so that it's obvious a few years from now. Thanks. We want to transition away from using nsIntSize and gfxIntSize and instead use gfx::IntSize or more specific size classes (like ScreenIntSize) when possible. https://hg.mozilla.org/integration/mozilla-inbound/rev/bad6c063b01f
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/6e221af28de2 for breaking the build for at least Windows: https://tbpl.mozilla.org/php/getParsedLog.php?id=30097699&tree=Mozilla-Inbound and B2G desktop Windows: https://tbpl.mozilla.org/php/getParsedLog.php?id=30097498&tree=Mozilla-Inbound
Hmm, the non-Windows, non-b2g-desktop-Windows failures actually parsed out an error: https://tbpl.mozilla.org/php/getParsedLog.php?id=30098888&tree=Mozilla-Inbound ../../../gecko/gfx/layers/ipc/ShadowLayerUtilsGralloc.cpp:497: error: conversion from 'const mozilla::gfx::IntSizeTyped<mozilla::gfx::UnknownUnits>' to non-scalar type 'gfxIntSize' requested ../../../gecko/gfx/layers/ipc/ShadowLayerUtilsGralloc.cpp:535: error: no match for 'operator=' in '* aSize = ((const mozilla::layers::SurfaceDescriptorGralloc*)((const mozilla::layers::SurfaceDescriptor*)aDescriptor)->mozilla::layers::SurfaceDescriptor::get_SurfaceDescriptorGralloc())->mozilla::layers::SurfaceDescriptorGralloc::size()' make[6]: *** [ShadowLayerUtilsGralloc.o] Error 1 make[5]: *** [gfx/layers/compile] Error 2 make[5]: *** Waiting for unfinished jobs.... make[4]: *** [compile] Error 2 make[3]: *** [default] Error 2 make[2]: *** [realbuild] Error 2 make[1]: *** [build] Error 2 make: *** [out/target/product/generic/obj/DATA/gecko_intermediates/gecko] Error 2 Return code: 2 failed to build
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•