[Blimp Client] Fixes up URL before sending it to Engine.

blimp/engine/session/blimp_engine_session.cc

 // 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.
 
 #include "blimp/engine/session/blimp_engine_session.h"
 
 #include <string>
 
 #include "base/strings/utf_string_conversions.h"
 #include "base/thread_task_runner_handle.h"
@@ skipping 302 matching lines @@
   }
 
   if (web_contents_ && web_contents_->GetRenderViewHost() &&
       web_contents_->GetRenderViewHost()->GetWidget()) {
     web_contents_->GetRenderViewHost()->GetWidget()->WasResized();
   }
 }
 
 void BlimpEngineSession::LoadUrl(const int target_tab_id, const GURL& url) {
   DVLOG(1) << "Load URL " << url << " in tab " << target_tab_id;
-  if (url.is_empty()) {
+  if (!url.is_valid()) {
+    VLOG(1) << "Dropping invalid URL " << url;
     return;
   }
 
   // TODO(dtrainor, haibinlu): Fix up the URL with url_fixer.h.  If that doesn't

[Kevin M, 2016/04/07 20:12:30]

Remove this comment

   // produce a valid spec() then try to build a search query?
   content::NavigationController::LoadURLParams params(url);
   params.transition_type = ui::PageTransitionFromInt(
       ui::PAGE_TRANSITION_TYPED | ui::PAGE_TRANSITION_FROM_ADDRESS_BAR);
   web_contents_->GetController().LoadURLWithParams(params);
   web_contents_->Focus();
 }
 
 void BlimpEngineSession::GoBack(const int target_tab_id) {
   DVLOG(1) << "Back in tab " << target_tab_id;
@@ skipping 266 matching lines @@
 
   aura::Window* parent = window_tree_host_->window();
   aura::Window* content = web_contents_->GetNativeView();
   if (!parent->Contains(content))
     parent->AddChild(content);
   content->Show();
 }
 
 }  // namespace engine
 }  // namespace blimp