Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wr2 #319

Merged
merged 33 commits into from Aug 3, 2016
Merged

Wr2 #319

merged 33 commits into from Aug 3, 2016

Conversation

@glennw
Copy link
Member

glennw commented Aug 2, 2016

No description provided.

gw3583 and others added 30 commits Jun 11, 2016
@@ -130,15 +130,13 @@ pub struct ResourceCache {

pending_raster_jobs: Vec<GlyphRasterJob>,

white_image_id: TextureCacheItemId,
dummy_mask_image_id: TextureCacheItemId,
//white_image_id: TextureCacheItemId,

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Just remove

self.debug.add_text((tile_x0.0 as f32 + tile_x1.0 as f32) * 0.5,
(tile_y0.0 as f32 + tile_y1.0 as f32) * 0.5,
label,
c);//&ColorF::new(0.0, 0.0, 0.0, 1.0));

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Remove commented out code here and above


self.update_shaders();
//self.update_shaders();

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Remove commented out stuff here

Some(self.render())
} else {
None
}
});
// hprof::end_frame();
//hprof::profiler().print_timing();

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Remove commented out code

@@ -196,27 +199,37 @@ impl RenderBackend {
self.build_scene();
self.render()
});
// hprof::end_frame();
//hprof::profiler().print_timing();

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Remove commented out code

/* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Delete this file?

pub fn finalize(&mut self, scrolling: &ScrollingState) {
self.scrolling = *scrolling;
self.aabb_tree.finalize();
//self.aabb_tree.finalize();

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Just delete this line

@@ -105,6 +73,7 @@ impl Layer {
pub fn print(&self) {
self.aabb_tree.print(NodeIndex(0), 0);
}
*/

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Remove this commented out code

src/lib.rs Outdated
@@ -99,6 +101,7 @@ extern crate app_units;
extern crate euclid;
extern crate fnv;
extern crate gleam;
//extern crate hprof;

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Remove this

@@ -0,0 +1,222 @@
/* This Source Code Form is subject to the terms of the Mozilla Public

This comment has been minimized.

@pcwalton

pcwalton Aug 2, 2016

Collaborator

Just delete this file

@pcwalton
Copy link
Collaborator

pcwalton commented Aug 2, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2016

📌 Commit 9b552f5 has been approved by pcwalton

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2016

Testing commit 9b552f5 with merge 0d27ab1...

bors-servo added a commit that referenced this pull request Aug 2, 2016
Wr2
@pcwalton
Copy link
Collaborator

pcwalton commented Aug 2, 2016

Here we go!

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2016

💔 Test failed - travis

@nox
Copy link
Member

nox commented Aug 3, 2016

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Aug 3, 2016

Testing commit 9b552f5 with merge 9db67b8...

bors-servo added a commit that referenced this pull request Aug 3, 2016
Wr2
@bors-servo
Copy link
Contributor

bors-servo commented Aug 3, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 9b552f5 into servo:master Aug 3, 2016
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
homu Test successful
Details
@liigo
Copy link

liigo commented Aug 4, 2016

@glennw @pcwalton

IMO, This PR has bad title (one word 'Wr2') and bad description (nothing). Most commits has no detail message. It's unfortunate for such a big PR.

@glennw glennw deleted the glennw:wr2 branch Dec 12, 2016
zakorgy added a commit to zakorgy/webrender that referenced this pull request Oct 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
You can’t perform that action at this time.