[Blimp] Refactor Blimp test engine with embedded test server and URL rewriting

blimp/engine/testing/session/blimp_engine_testing_session.h

+// Copyright 2016 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 BLIMP_ENGINE_TESTING_SESSION_BLIMP_ENGINE_TESTING_SESSION_H_
+#define BLIMP_ENGINE_TESTING_SESSION_BLIMP_ENGINE_TESTING_SESSION_H_
+
+#include "blimp/common/proto/blimp_message.pb.h"
+#include "blimp/engine/common/blimp_browser_context.h"
+#include "blimp/engine/feature/engine_settings_feature.h"
+#include "blimp/engine/session/blimp_engine_session.h"
+#include "blimp/net/blimp_message_processor.h"
+#include "content/public/browser/browser_url_handler.h"

[Kevin M, 2017/01/09 20:17:18]

Not used?

[shenghuazhang, 2017/01/10 23:20:50]

Done.

+#include "content/public/browser/web_contents_delegate.h"
+#include "net/base/completion_callback.h"
+
+namespace blimp {
+namespace engine {
+
+class BlimpEngineTestingSession : public BlimpEngineSession {

[Kevin M, 2017/01/09 20:17:18]

nit: This should be implemented as a BlimpMessage rewriting filter instead of as
BlimpEngineSession. No need to fix this, it's too major, but IMO this is change
is too invasive for the task this class accomplishes.

[shenghuazhang, 2017/01/10 23:20:50]

Acknowledged.

+ public:
+  BlimpEngineTestingSession(
+      std::unique_ptr<BlimpBrowserContext> browser_context,
+      net::NetLog* net_log,
+      BlimpEngineConfig* config,
+      SettingsManager* settings_manager);
+  ~BlimpEngineTestingSession() = default;
+
+  // Override BlimpEngineSession::BlimpEngineSession
+  // This object handles NavigationMessage::LOAD_URL
+  // url rewritting with testing url handler
+  void ProcessMessage(std::unique_ptr<BlimpMessage> message,
+                      const net::CompletionCallback& callback) override;
+};
+
+}  // namespace engine
+}  // namespace blimp
+
+#endif  // BLIMP_ENGINE_TESTING_SESSION_BLIMP_ENGINE_TESTING_SESSION_H_