[Presentation API / Media Router] Clean up default pres URL logic.

chrome/browser/media/router/media_router_dialog_controller.h

Index: chrome/browser/media/router/media_router_dialog_controller.h
diff --git a/chrome/browser/media/router/media_router_dialog_controller.h b/chrome/browser/media/router/media_router_dialog_controller.h
index 40dce80fa45b6e1db5ecf6464c66bc18da54ca24..340fdc6098983042a87786acb00d9be080629e72 100644
--- a/chrome/browser/media/router/media_router_dialog_controller.h
+++ b/chrome/browser/media/router/media_router_dialog_controller.h
@@ -67,8 +67,8 @@ class MediaRouterDialogController {
 
   // Returns the CreatePresentationSessionRequest to the caller but keeps the
   // ownership with the MediaRouterDialogController.
-  const CreatePresentationSessionRequest* presentation_request() const {
-    return presentation_request_.get();
+  const CreatePresentationSessionRequest* create_session_request() const {

[mlamouri (slow - plz ping), 2015/10/27 18:42:51]

That name doesn't look like a getter but like a function. I think the object and
member names need to change.

[imcheng, 2015/10/27 20:50:14]

The fact that the method name is in unix hacker style should suggest it is a
getter. That said, I do see that it can be a little confusing for the class name
/ member to start with a verb. Suggestion for a new name is welcome.

+    return create_session_request_.get();
   }
 
   // Returns the WebContents that initiated showing the dialog.
@@ -94,7 +94,7 @@ class MediaRouterDialogController {
   // Data for dialogs created at the request of the Presentation API.
   // Passed from the caller via ShowMediaRouterDialogForPresentation to the
   // dialog when it is initialized.
-  scoped_ptr<CreatePresentationSessionRequest> presentation_request_;
+  scoped_ptr<CreatePresentationSessionRequest> create_session_request_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaRouterDialogController);
 };