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

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java

Issue 2440643002: Remove TabModelSelector's dependency on ChromeFullscreenManager. (Closed)
Patch Set: file move Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.webapps; 5 package org.chromium.chrome.browser.webapps;
6 6
7 import android.content.Intent; 7 import android.content.Intent;
8 import android.graphics.Bitmap; 8 import android.graphics.Bitmap;
9 import android.graphics.Color; 9 import android.graphics.Color;
10 import android.graphics.drawable.Drawable; 10 import android.graphics.drawable.Drawable;
(...skipping 551 matching lines...) Expand 10 before | Expand all | Expand 10 after
562 562
563 @VisibleForTesting 563 @VisibleForTesting
564 boolean isUrlBarVisible() { 564 boolean isUrlBarVisible() {
565 return findViewById(R.id.control_container).getVisibility() == View.VISI BLE; 565 return findViewById(R.id.control_container).getVisibility() == View.VISI BLE;
566 } 566 }
567 567
568 @Override 568 @Override
569 protected ChromeFullscreenManager createFullscreenManager() { 569 protected ChromeFullscreenManager createFullscreenManager() {
570 return new ChromeFullscreenManager(this, 570 return new ChromeFullscreenManager(this,
571 (ControlContainer) findViewById(R.id.control_container), 571 (ControlContainer) findViewById(R.id.control_container),
572 getControlContainerHeightResource(), false /* supportsBrowserOve rride */); 572 getTabModelSelector(), getControlContainerHeightResource(),
573 false /* supportsBrowserOverride */);
573 } 574 }
574 575
575 @Override 576 @Override
576 public int getControlContainerHeightResource() { 577 public int getControlContainerHeightResource() {
577 return R.dimen.webapp_control_container_height; 578 return R.dimen.webapp_control_container_height;
578 } 579 }
579 580
580 @Override 581 @Override
581 protected Drawable getBackgroundDrawable() { 582 protected Drawable getBackgroundDrawable() {
582 return null; 583 return null;
583 } 584 }
584 585
585 @Override 586 @Override
586 protected TabDelegateFactory createTabDelegateFactory() { 587 protected TabDelegateFactory createTabDelegateFactory() {
587 return new WebappDelegateFactory(this); 588 return new WebappDelegateFactory(this);
588 } 589 }
589 590
590 // We're temporarily disable CS on webapp since there are some issues. (http ://crbug.com/471950) 591 // We're temporarily disable CS on webapp since there are some issues. (http ://crbug.com/471950)
591 // TODO(changwan): re-enable it once the issues are resolved. 592 // TODO(changwan): re-enable it once the issues are resolved.
592 @Override 593 @Override
593 protected boolean isContextualSearchAllowed() { 594 protected boolean isContextualSearchAllowed() {
594 return false; 595 return false;
595 } 596 }
596 } 597 }
OLDNEW
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/webapps/FullScreenActivity.java ('k') | chrome/android/java_sources.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698