Merge lp:~micahg/xulrunner/xul-1.9.1.head.fixlp327863 into lp:~mozillateam/xulrunner/xulrunner-1.9.1.head

Proposed by Micah Gersten
Status: Needs review
Proposed branch: lp:~micahg/xulrunner/xul-1.9.1.head.fixlp327863
Merge into: lp:~mozillateam/xulrunner/xulrunner-1.9.1.head
Diff against target: 48 lines
2 files modified
debian/patches/bz486065_att388372_fix_gtk_borders.patch (+35/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~micahg/xulrunner/xul-1.9.1.head.fixlp327863
To post a comment you must log in.
Revision history for this message
Micah Gersten (micahg) wrote :

Patch built successfully in my ppa. Tested by LLStarks.

Unmerged revisions

500. By Micah Gersten

 * fix LP: #327863
   - added debian/patches/bz486065_att388372_fix_gtk_borders.patch
   - updated debian/patches/series

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'debian/patches/bz486065_att388372_fix_gtk_borders.patch'
2--- debian/patches/bz486065_att388372_fix_gtk_borders.patch 1970-01-01 00:00:00 +0000
3+++ debian/patches/bz486065_att388372_fix_gtk_borders.patch 2009-09-30 01:50:20 +0000
4@@ -0,0 +1,35 @@
5+--- a/layout/generic/nsHTMLReflowState.cpp Wed Jul 29 18:10:50 2009 -0400
6++++ b/layout/generic/nsHTMLReflowState.cpp Thu Jul 30 15:43:22 2009 +1200
7+@@ -1925,7 +1925,8 @@ nsCSSOffsetState::InitOffsets(nscoord aC
8+ }
9+ mComputedBorderPadding += mComputedPadding;
10+
11+- if (frame->GetType() == nsGkAtoms::tableFrame) {
12++ nsIAtom* frameType = frame->GetType();
13++ if (frameType == nsGkAtoms::tableFrame) {
14+ nsTableFrame *tableFrame = static_cast<nsTableFrame*>(frame);
15+
16+ if (tableFrame->IsBorderCollapse()) {
17+@@ -1936,6 +1937,23 @@ nsCSSOffsetState::InitOffsets(nscoord aC
18+ mComputedPadding.SizeTo(0,0,0,0);
19+ mComputedBorderPadding = tableFrame->GetIncludedOuterBCBorder();
20+ }
21++ } else if (frameType == nsGkAtoms::scrollbarFrame) {
22++ // scrollbars may have had their width or height smashed to zero
23++ // by the associated scrollframe, in which case we must not report
24++ // any padding or border on that axis.
25++ nsSize size(frame->GetSize());
26++ if (size.width == 0) {
27++ mComputedPadding.left = 0;
28++ mComputedPadding.right = 0;
29++ mComputedBorderPadding.left = 0;
30++ mComputedBorderPadding.right = 0;
31++ }
32++ if (size.height == 0) {
33++ mComputedPadding.top = 0;
34++ mComputedPadding.bottom = 0;
35++ mComputedBorderPadding.top = 0;
36++ mComputedBorderPadding.bottom = 0;
37++ }
38+ }
39+ }
40
41=== modified file 'debian/patches/series'
42--- debian/patches/series 2009-09-24 00:42:26 +0000
43+++ debian/patches/series 2009-09-30 01:50:20 +0000
44@@ -18,3 +18,4 @@
45 bzXXX_fix_python_dom_pyxultest.patch
46 testsuite_skip_tests_depending_on_utf8.patch
47 bz473629_lp319480_leak_setenv_apprunner.patch
48+bz486065_att388372_fix_gtk_borders.patch

Subscribers

People subscribed via source and target branches