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

chrome/browser/media/router/presentation_request.h

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_MEDIA_ROUTER_PRESENTATION_REQUEST_H_
+#define CHROME_BROWSER_MEDIA_ROUTER_PRESENTATION_REQUEST_H_
+
+#include <string>
+
+#include "chrome/browser/media/router/render_frame_host_id.h"
+#include "url/gurl.h"
+
+namespace media_router {
+
+// Represents a presentation request made from a render frame. Contains the
+// presentation URL of the request, and information on the originating frame.
+class PresentationRequest {
+ public:
+  PresentationRequest(const RenderFrameHostId& render_frame_host_id,
+                      const std::string& presentation_url,
+                      const GURL& frame_url);
+  ~PresentationRequest();
+
+  bool Equals(const PresentationRequest& other) const;
+
+  const RenderFrameHostId& render_frame_host_id() const {
+    return render_frame_host_id_;
+  }
+  const std::string& presentation_url() const { return presentation_url_; }
+  const GURL& frame_url() const { return frame_url_; }
+
+ private:
+  // ID of RenderFrameHost that initiated the request.
+  const RenderFrameHostId render_frame_host_id_;
+
+  // URL of default presentation.

[mark a. foltz, 2015/10/20 20:15:25]

Can this be any presentation URL?

[imcheng, 2015/10/24 00:41:19]

Yes. Removed "default".

+  const std::string presentation_url_;
+
+  // URL of frame from which the request was initiated.
+  const GURL frame_url_;

[mark a. foltz, 2015/10/20 20:15:25]

What is this used for?  The frame may have no src=, or may have navigated, etc.
after issuing the request.

[imcheng, 2015/10/24 00:41:19]

This is for matching the the default request that was in the dialog with the one
set in PSDImpl. All of (RFH id, pres url, frame url) has to match in order to
trigger default request.

+};
+
+}  // namespace media_router
+
+#endif  // CHROME_BROWSER_MEDIA_ROUTER_PRESENTATION_REQUEST_H_