Commit ea800be6 authored by dtapuska's avatar dtapuska Committed by Commit bot

Mojo HTML viewer crash

Check that we have a valid navigator_host_ object before we call methods on it

BUG=443193

Review URL: https://codereview.chromium.org/790793007

Cr-Commit-Position: refs/heads/master@{#310090}
parent 78b59cef
......@@ -243,9 +243,11 @@ blink::WebNavigationPolicy HTMLDocument::decidePolicyForNavigation(
if (CanNavigateLocally(frame, request))
return default_policy;
navigator_host_->RequestNavigate(
WebNavigationPolicyToNavigationTarget(default_policy),
mojo::URLRequest::From(request).Pass());
if (navigator_host_.get()) {
navigator_host_->RequestNavigate(
WebNavigationPolicyToNavigationTarget(default_policy),
mojo::URLRequest::From(request).Pass());
}
return blink::WebNavigationPolicyIgnore;
}
......@@ -261,7 +263,8 @@ void HTMLDocument::didNavigateWithinPage(
blink::WebLocalFrame* frame,
const blink::WebHistoryItem& history_item,
blink::WebHistoryCommitType commit_type) {
navigator_host_->DidNavigateLocally(history_item.urlString().utf8());
if (navigator_host_.get())
navigator_host_->DidNavigateLocally(history_item.urlString().utf8());
}
blink::WebEncryptedMediaClient* HTMLDocument::encryptedMediaClient() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment