Use alt DDG logo when height is prohibitively small to use main one

app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt

@@ -22,6 +22,7 @@ import android.arch.lifecycle.Observer
 import android.arch.lifecycle.ViewModelProviders
 import android.content.Context
 import android.content.Intent
+import android.content.res.Configuration
 import android.net.Uri
 import android.os.Bundle
 import android.support.annotation.StringRes
@@ -611,6 +612,14 @@ class BrowserTabFragment : Fragment(), FindListener {
         }
     }
 
+    /**
+     * We don't destroy the activity on config changes like orientation, so we need to ensure we update resources which might change based on config
+     */
+    override fun onConfigurationChanged(newConfig: Configuration) {
+        super.onConfigurationChanged(newConfig)
+        ddgLogo.setImageResource(R.drawable.full_logo)
+    }
+
     private fun resetTabState() {
         omnibarTextInput.text.clear()
         viewModel.resetView()

app/src/main/res/layout/fragment_browser_tab.xml

@@ -34,26 +34,22 @@
             android:id="@+id/logoParent"
             android:layout_width="match_parent"
             android:layout_height="match_parent"
+            android:layout_marginTop="?attr/actionBarSize"
             android:animateLayoutChanges="true">
 
-            <android.support.constraint.Guideline
-                android:id="@+id/guidelineTop"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:orientation="horizontal"
-                app:layout_constraintGuide_percent="0.5" />
-
             <ImageView
                 android:id="@+id/ddgLogo"
-                android:layout_width="180dp"
-                android:layout_height="180dp"
+                android:layout_width="0dp"
+                android:layout_height="0dp"
                 android:contentDescription="@string/duckDuckGoLogoDescription"
                 android:src="@drawable/full_logo"
-                app:layout_constraintBottom_toBottomOf="@id/guidelineTop"
+                app:layout_constraintBottom_toBottomOf="parent"
                 app:layout_constraintEnd_toEndOf="parent"
+                app:layout_constraintHeight_max="180dp"
                 app:layout_constraintStart_toStartOf="parent"
-                app:layout_constraintTop_toTopOf="@id/guidelineTop"
-                />
+                app:layout_constraintTop_toTopOf="parent"
+                app:layout_constraintVertical_bias="0.5"
+                app:layout_constraintWidth_max="180dp" />
 
         </android.support.constraint.ConstraintLayout>