Skip to content

Commit

Permalink
Fix regression on clicking links
Browse files Browse the repository at this point in the history
  • Loading branch information
Eric Atkinson committed Jul 17, 2013
1 parent c17ede3 commit 9c5ab15
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions src/components/main/layout/layout_task.rs
Expand Up @@ -241,8 +241,8 @@ impl LayoutTask {

// TODO: Set options on the builder before building.
// TODO: Be smarter about what needs painting.
for layout_root.traverse_preorder |flow| {
flow.build_display_list(&builder, &layout_root.position(), display_list);
do layout_root.partially_traverse_preorder |flow| {
flow.build_display_list(&builder, &layout_root.position(), display_list)
}

let root_size = do layout_root.with_base |base| {
Expand Down Expand Up @@ -349,9 +349,13 @@ impl LayoutTask {
};
let display_list: @Cell<DisplayList<RenderBox>> =
@Cell::new(DisplayList::new());
flow.build_display_list(&builder,
&flow.position(),
display_list);

do flow.partially_traverse_preorder |this_flow| {
this_flow.build_display_list(&builder,
&flow.position(),
display_list)

}
let (x, y) = (Au::from_frac_px(point.x as float),
Au::from_frac_px(point.y as float));
let mut resp = Err(());
Expand Down

5 comments on commit 9c5ab15

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from jdm
at eric93@9c5ab15

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging eric93/servo/diplaylist-refactor = 9c5ab15 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eric93/servo/diplaylist-refactor = 9c5ab15 merged ok, testing candidate = e5fc648

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = e5fc648

Please sign in to comment.