Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(738)

Unified Diff: chrome/browser/ui/webui/ntp/android/new_tab_page_ready_handler.cc

Issue 259393006: Remove WebUI NTP on Android. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/ntp/android/new_tab_page_ready_handler.cc
diff --git a/chrome/browser/ui/webui/ntp/android/new_tab_page_ready_handler.cc b/chrome/browser/ui/webui/ntp/android/new_tab_page_ready_handler.cc
deleted file mode 100644
index 01fd6b8f8d0c73f958992bf4906253e584758bfd..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/webui/ntp/android/new_tab_page_ready_handler.cc
+++ /dev/null
@@ -1,52 +0,0 @@
-// Copyright (c) 2012 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 "chrome/browser/ui/webui/ntp/android/new_tab_page_ready_handler.h"
-
-#include "base/bind.h"
-#include "base/logging.h"
-#include "base/values.h"
-#include "chrome/browser/android/tab_android.h"
-#include "content/public/browser/web_contents.h"
-#include "content/public/browser/web_ui.h"
-
-NewTabPageReadyHandler::NewTabPageReadyHandler() {
-}
-
-NewTabPageReadyHandler::~NewTabPageReadyHandler() {
-}
-
-void NewTabPageReadyHandler::RegisterMessages() {
- web_ui()->RegisterMessageCallback("notifyNTPReady", base::Bind(
- &NewTabPageReadyHandler::HandleNewTabPageReady, base::Unretained(this)));
- web_ui()->RegisterMessageCallback("NTPUnexpectedNavigation", base::Bind(
- &NewTabPageReadyHandler::HandleNewTabPageUnexpectedNavigation,
- base::Unretained(this)));
- web_ui()->RegisterMessageCallback("notifyNTPTitleLoaded", base::Bind(
- &NewTabPageReadyHandler::HandleNewTabPageTitleLoaded,
- base::Unretained(this)));
-}
-
-void NewTabPageReadyHandler::HandleNewTabPageTitleLoaded(
- const base::ListValue* args) {
- web_ui()->OverrideTitle(base::string16());
-}
-
-void NewTabPageReadyHandler::HandleNewTabPageReady(
- const base::ListValue* args) {
- TabAndroid* tab = TabAndroid::FromWebContents(web_ui()->GetWebContents());
- if (!tab)
- return;
- tab->OnNewTabPageReady();
-}
-
-void NewTabPageReadyHandler::HandleNewTabPageUnexpectedNavigation(
- const base::ListValue* args) {
- // NTP reached an unexpected state trying to send finish loading notification
- // a second time. The notification should be sent only when page is
- // completely done loading. This could otherwise create a race condition in
- // tests waiting for the NTP to have loaded (any navigation NTP does after
- // loading could interfere with the test navigation).
- NOTREACHED();
-}

Powered by Google App Engine
This is Rietveld 408576698