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

Side by Side Diff: content/public/android/java/src/org/chromium/content/browser/ContentReadbackHandler.java

Issue 1021863005: Pass format as argument to getContentBitmapAsync. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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.content.browser; 5 package org.chromium.content.browser;
6 6
7 import android.graphics.Bitmap; 7 import android.graphics.Bitmap;
8 import android.graphics.Rect; 8 import android.graphics.Rect;
9 import android.util.SparseArray; 9 import android.util.SparseArray;
10 10
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 * Asynchronously, generate and grab a bitmap representing what is currently on the screen 74 * Asynchronously, generate and grab a bitmap representing what is currently on the screen
75 * for {@code view}. 75 * for {@code view}.
76 * 76 *
77 * @param scale The scale that should be applied to the content. 77 * @param scale The scale that should be applied to the content.
78 * @param srcRect A subrect of the original content to capture. If this is empty, it will grab 78 * @param srcRect A subrect of the original content to capture. If this is empty, it will grab
79 * the whole surface. 79 * the whole surface.
80 * @param view The {@link ContentViewCore} to grab the bitmap from. 80 * @param view The {@link ContentViewCore} to grab the bitmap from.
81 * @param callback The callback to be executed after readback completes. 81 * @param callback The callback to be executed after readback completes.
82 */ 82 */
83 public void getContentBitmapAsync(float scale, Rect srcRect, ContentViewCore view, 83 public void getContentBitmapAsync(float scale, Rect srcRect, ContentViewCore view,
84 GetBitmapCallback callback) { 84 Bitmap.Config config, GetBitmapCallback callback) {
Yaron 2015/03/25 14:13:56 You need to update the javadoc accordingly
85 if (!readyForReadback()) { 85 if (!readyForReadback()) {
86 callback.onFinishGetBitmap(null); 86 callback.onFinishGetBitmap(null);
87 return; 87 return;
88 } 88 }
89 ThreadUtils.assertOnUiThread(); 89 ThreadUtils.assertOnUiThread();
90 90
91 int readbackId = mNextReadbackId++; 91 int readbackId = mNextReadbackId++;
92 mGetBitmapRequests.put(readbackId, callback); 92 mGetBitmapRequests.put(readbackId, callback);
93 nativeGetContentBitmap(mNativeContentReadbackHandler, readbackId, scale, 93 nativeGetContentBitmap(mNativeContentReadbackHandler, readbackId, scale,
94 Bitmap.Config.ARGB_8888, srcRect.top, srcRect.left, srcRect.widt h(), 94 config, srcRect.top, srcRect.left, srcRect.width(),
95 srcRect.height(), view); 95 srcRect.height(), view);
96 } 96 }
97 97
98 /** 98 /**
99 * Asynchronously, grab a bitmap of the current browser compositor root laye r. 99 * Asynchronously, grab a bitmap of the current browser compositor root laye r.
100 * 100 *
101 * @param windowAndroid The window that hosts the compositor. 101 * @param windowAndroid The window that hosts the compositor.
102 * @param callback The callback to be executed after readback completes . 102 * @param callback The callback to be executed after readback completes .
103 */ 103 */
104 public void getCompositorBitmapAsync(WindowAndroid windowAndroid, GetBitmapC allback callback) { 104 public void getCompositorBitmapAsync(WindowAndroid windowAndroid, GetBitmapC allback callback) {
(...skipping 16 matching lines...) Expand all
121 protected abstract boolean readyForReadback(); 121 protected abstract boolean readyForReadback();
122 122
123 private native long nativeInit(); 123 private native long nativeInit();
124 private native void nativeDestroy(long nativeContentReadbackHandler); 124 private native void nativeDestroy(long nativeContentReadbackHandler);
125 private native void nativeGetContentBitmap(long nativeContentReadbackHandler , int readbackId, 125 private native void nativeGetContentBitmap(long nativeContentReadbackHandler , int readbackId,
126 float scale, Bitmap.Config config, float x, float y, float width, fl oat height, 126 float scale, Bitmap.Config config, float x, float y, float width, fl oat height,
127 Object contentViewCore); 127 Object contentViewCore);
128 private native void nativeGetCompositorBitmap(long nativeContentReadbackHand ler, 128 private native void nativeGetCompositorBitmap(long nativeContentReadbackHand ler,
129 int readbackId, long nativeWindowAndroid); 129 int readbackId, long nativeWindowAndroid);
130 } 130 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698