aboutsummaryrefslogtreecommitdiffstats
path: root/components/webdriver_server
diff options
context:
space:
mode:
authorAneesh Agrawal <aneeshusa@gmail.com>2016-06-09 07:01:24 -0400
committerAneesh Agrawal <aneeshusa@gmail.com>2016-09-13 15:17:40 -0400
commit9d097e7d1570d68e5fe77ff01595cf14a3e2cc29 (patch)
tree32e6f49e53774b2fdfb4dec38223d48f9b12a7f9 /components/webdriver_server
parentbb53da69578887befae195b18255967cc7f3fc59 (diff)
downloadservo-9d097e7d1570d68e5fe77ff01595cf14a3e2cc29.tar.gz
servo-9d097e7d1570d68e5fe77ff01595cf14a3e2cc29.zip
Use fn pipeline_id consistently, not fn pipeline
Consistently use the name 'pipeline_id' to refer to a function that returns an (optional) PipelineId. This was prompted by discovering both fn pipeline and fn pipeline_id doing the same job in htmliframeelement.rs. Note that there is fn pipeline in components/compositing/compositor.rs, but that actually returns an Option<&CompositionPipeline>, not any kind of PipelineId.
Diffstat (limited to 'components/webdriver_server')
-rw-r--r--components/webdriver_server/lib.rs6
1 files changed, 3 insertions, 3 deletions
diff --git a/components/webdriver_server/lib.rs b/components/webdriver_server/lib.rs
index 3206af03a32..48f4c0ad1ee 100644
--- a/components/webdriver_server/lib.rs
+++ b/components/webdriver_server/lib.rs
@@ -251,7 +251,7 @@ impl Handler {
}
}
- fn pipeline(&self, frame_id: Option<FrameId>) -> WebDriverResult<PipelineId> {
+ fn pipeline_id(&self, frame_id: Option<FrameId>) -> WebDriverResult<PipelineId> {
let interval = 20;
let iterations = 30_000 / interval;
let (sender, receiver) = ipc::channel().unwrap();
@@ -271,11 +271,11 @@ impl Handler {
}
fn root_pipeline(&self) -> WebDriverResult<PipelineId> {
- self.pipeline(None)
+ self.pipeline_id(None)
}
fn frame_pipeline(&self) -> WebDriverResult<PipelineId> {
- self.pipeline(self.session.as_ref().and_then(|session| session.frame_id))
+ self.pipeline_id(self.session.as_ref().and_then(|session| session.frame_id))
}
fn session(&self) -> WebDriverResult<&WebDriverSession> {